Age | Commit message (Collapse) | Author | Files | Lines |
|
svn path=/trunk/; revision=52603
|
|
protocol code
svn path=/trunk/; revision=52602
|
|
svn path=/trunk/; revision=52601
|
|
time in order to fill in their respective context menus. R52386 broke
that. Fix it and add comments about handling reparenting. Fixes bug
9265.
svn path=/trunk/; revision=52600
|
|
don't assign the output of pntoh24() to a gint16
unfortunately, vwr detection does not work reliably and many pdf files
are recognized as vwr - this commit should prevent wireshark from
crashing when it tries to load the USB 2.0 spec as pdf ;-)
svn path=/trunk/; revision=52599
|
|
- fix a typo
- Add the library dirs and not just the libraries.
This would help if the paths that are now returned
were actually correct.
svn path=/trunk/; revision=52598
|
|
(http://www.wireshark.org/lists/wireshark-dev/201310/msg00159.html) :
Since r49923, TCP flags are no more displayed in info column when tree is NULL
svn path=/trunk/; revision=52597
|
|
(https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9208).
1. Fix ECDH server key exchange for TLS 1.2
2. Fix RSA server key exchange for TLS 1.2
From Hauke Mehrtens
svn path=/trunk/; revision=52596
|
|
9208 (https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9208)
1. Fix ECDH server key exchange for DTLS 1.2
2. Fix RSA server key exchange for DTLS 1.2
From Hauke Mehrtens
svn path=/trunk/; revision=52595
|
|
svn path=/trunk/; revision=52594
|
|
per protocol version.
svn path=/trunk/; revision=52593
|
|
Retransmitted responses not only add additional (bogus) transactions but also the latency associated with them. This can greatly inflate the maximum and average SRT stats especially in the case of retransmissions triggered by the expiry of the rexmit timer (RTOs). Only calculating SRT for the first response received accomplishes this goal without requiring the TCP pref "Do not call subdissectors for error packets" to be set.
svn path=/trunk/; revision=52592
|
|
svn path=/trunk/; revision=52591
|
|
svn path=/trunk/; revision=52590
|
|
svn path=/trunk/; revision=52589
|
|
svn path=/trunk/; revision=52588
|
|
as guint32 internally - they may be signed and/or 64-bit.
svn path=/trunk/; revision=52587
|
|
svn path=/trunk/; revision=52584
|
|
anyways, since Michael made key operations non-destructive in r44380.
svn path=/trunk/; revision=52583
|
|
9261 (https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9261).
From Roland Knall
svn path=/trunk/; revision=52582
|
|
svn path=/trunk/; revision=52581
|
|
using a Visual Studio solution.
Add CMake properties to group the source files in epan into logical blocks when using a Visual Studio solution.
svn path=/trunk/; revision=52580
|
|
svn path=/trunk/; revision=52579
|
|
proto_tree_add_item was valid *before* we short-circuited based on a NULL tree.
This was good in that it removed a common source of really-long-loop bugs. It
was less good in that it cost us about 8% in speed when doing a tree-less
dissection, but we decided the tradeoff was worth it.
After Anders' recent mail to -dev about performance, I started thinking about
how to optimize this. It occurred to me that the vast majority of the logic
involved in the check was dealing with the length value - fetching the actual
length if it was a counted string, calculating the length if it was -1, adding
the length to the offset in a way that was free from overflows, etc.
All of this is (theoretically) unnecessary - simply checking the offset without
worrying about the length will still catch the very-long-loops, since it is the
offset that increases in each iteration, not the length.
All that to justify:
- implement tvb_ensure_offset_exists which throws an exception if the offset is
not within the tvb
- use it instead of all the complicated other logic in the pre-short-circuit
step of proto_tree_add_item and friends
This gives us back about 3/4 of the performance we lost in the original patch.
We're still ~2% slower than without any check, but this is the best I can think
of right now.
svn path=/trunk/; revision=52578
|
|
tvb->reported_length. Small performance win on the hot path, and a bit easier to
read.
svn path=/trunk/; revision=52577
|
|
svn path=/trunk/; revision=52576
|
|
dissecting without tree, they are costly because they now happen for every
proto_tree_add_item call even if tree is NULL.
svn path=/trunk/; revision=52575
|
|
problem/solution.
"Long" since solved by copying the solution from Makefile.nmake.
Small update to the status of cmake builds on windows
svn path=/trunk/; revision=52574
|
|
tree_data and reference it directly when allocating/freeing tree items. This
lets us keep multiple around when we need them, and still lets us use
wmem_free_all for a major speedup. It also, coincidentally, lets us get rid of
the annoying fi_tmp hack that was needed before, since that element gets swept
up in the free_all with everything else.
Keep one pool cached to avoid creating/destroying a pool for each packet,
another minor performance win.
The various changes in approach seem to balance out pretty much exactly, this
still gives ~11% over pre-52569.
svn path=/trunk/; revision=52573
|
|
at once.
svn path=/trunk/; revision=52572
|
|
tvb_reported_length_remaining() return 0 instead of -1. This should provide a better long-term fix for bugs/problems such as seen in https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9263. In the long-run, if we like this behavior, we can change the return value to a guint as well. Let's let it simmer for awhile first ...
svn path=/trunk/; revision=52571
|
|
always return a value >= 0. Part of fix for https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9263
#BACKPORT(1.10,1.8)
svn path=/trunk/; revision=52570
|
|
field_info separately. We still have to walk the tree in order to free certain
fvalues, but that's not a big deal. Another ~11% speed-up running "tshark -nVr"
on a large capture.
svn path=/trunk/; revision=52569
|
|
Even without making use of free_all (which should be possible) this still
results in ~8% speedup running "tshark -nVr" on a large file in my tests.
svn path=/trunk/; revision=52568
|
|
svn path=/trunk/; revision=52567
|
|
Fix dissection of PTP Management messages
svn path=/trunk/; revision=52566
|
|
instead. Fixes another one of the issues that shows up in
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9263
svn path=/trunk/; revision=52565
|
|
right way.
Fixes https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9256 without
reintroducing https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9212
svn path=/trunk/; revision=52564
|
|
Add a new strutil function format_text_chr that replaces unprintable characters
with a single passed-in char (eg a '.' or a '-') instead of a C-style escape.
This is useful for displaying binary data that frequently but not always
contains text; otherwise the number of C escape codes makes it unreadable.
svn path=/trunk/; revision=52563
|
|
gdk_pixbuf_new_from_inline().
svn path=/trunk/; revision=52562
|
|
svn path=/trunk/; revision=52561
|
|
a typedef of a *pointer* to the struct, not the struct itself, which are
different sizes.
This doesn't show up under valgrind because the length isn't checked in that
case, everything gets subsumed in valgrind's malloc/free hooks.
Fixes https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9264
svn path=/trunk/; revision=52560
|
|
init_progfile_dir(). Try using QCoreApplication::applicationFilePath()
instead.
svn path=/trunk/; revision=52559
|
|
svn path=/trunk/; revision=52558
|
|
svn path=/trunk/; revision=52557
|
|
svn path=/trunk/; revision=52556
|
|
(untested).
svn path=/trunk/; revision=52555
|
|
svn path=/trunk/; revision=52554
|
|
just call cf_open().
svn path=/trunk/; revision=52553
|
|
svn path=/trunk/; revision=52552
|