Re: [Wireshark-dev] Undissected packet bytes

2015-02-04 Thread Evan Huus
I think you will need to add a function very similar to proto_find_field_from_offset that makes use of proto_tree_traverse_pre_order and tracks which bytes have been seen as the traversal proceeds. On Wed, Feb 4, 2015 at 11:22 AM, Dario Lombardo dario.lombardo...@gmail.com wrote: On Wed, Feb 4,

Re: [Wireshark-dev] Undissected packet bytes

2015-02-04 Thread Dario Lombardo
On Wed, Feb 4, 2015 at 5:25 PM, Evan Huus eapa...@gmail.com wrote: I think you will need to add a function very similar to proto_find_field_from_offset that makes use of proto_tree_traverse_pre_order and tracks which bytes have been seen as the traversal proceeds. And how the output could

Re: [Wireshark-dev] Undissected packet bytes

2015-02-04 Thread Dario Lombardo
On Wed, Feb 4, 2015 at 3:52 PM, Evan Huus eapa...@gmail.com wrote: As a side note, I would expect that method to be *very* slow, since it traverses the entire tree for every byte of the packet. Traversing the tree once and maintaining a set of covered/uncovered ranges would be much more

[Wireshark-dev] Copying of filters from the filter tool bar with ctrl-c is broken

2015-02-04 Thread Anders Broman
Hi, Copying of filters from the filter tool bar with ctrl-c is broken, this commit is probably the culprit. https://code.wireshark.org/review/#/c/5832/ Regards Anders ___ Sent via:Wireshark-dev mailing list

Re: [Wireshark-dev] Undissected packet bytes

2015-02-04 Thread Michal Labedzki
I use expert info about undecoded thing in Bluetooth dissectors. In proto.h: /** The data is undecoded, the protocol dissection is incomplete here, usually PI_WARN severity */ #define PI_UNDECODED0x0500 But I use it with PI_NOTE, because I treat PI_WARN more like a bug rather

Re: [Wireshark-dev] Undissected packet bytes

2015-02-04 Thread Dario Lombardo
Hi Evan This looks promising. Where can I use this call? If I run it from within a dissector packet-hpfeeds.c:344 (just an example... for purpose of testing) for (i = 0; i len; i++) { fi = proto_find_field_from_offset(tree, i, tvb); g_print(TEST1 %p %d %p = %p\n,

Re: [Wireshark-dev] Undissected packet bytes

2015-02-04 Thread Dario Lombardo
That's a possibility (I have to take a look at the dissectors you both mentioned, and I didn't it yet), but what I was looking for was a more general way to do that. If the dissector itself has to make this checks, it means that all dissectors code must be patched. If we find a way to do that

Re: [Wireshark-dev] Undissected packet bytes

2015-02-04 Thread Evan Huus
I suppose you could add a an expert info, but I think that might be overkill, most users probably don't care that much. You could just log it, or dissect it as data, or... On Wed, Feb 4, 2015 at 11:58 AM, Dario Lombardo dario.lombardo...@gmail.com wrote: On Wed, Feb 4, 2015 at 5:25 PM, Evan Huus

Re: [Wireshark-dev] Copying of filters from the filter tool bar with ctrl-c is broken

2015-02-04 Thread Peter Wu
On 02/04/2015 05:17 PM, Anders Broman wrote: Hi, Copying of filters from the filter tool bar with ctrl-c is broken, this commit is probably the culprit. https://code.wireshark.org/review/#/c/5832/ Confirmed that modifiers are broken. Ctrl-A is also broken. Thanks for the head-ups, I'll have

[Wireshark-dev] Wireshark 1.99.2 is now available

2015-02-04 Thread Gerald Combs
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 I'm proud to announce the release of Wireshark 1.99.2. This is an experimental release intended to test new features for Wireshark 2.0. __ What is Wireshark? Wireshark

Re: [Wireshark-dev] [Wireshark-commits] master 7ced085: emem is dead! Long live wmem!

2015-02-04 Thread Maynard, Chris
Might that be a little hasty to remove the EMEM page so soon? I'm sure there are developers still working with older releases that are using emem. I don't know what information was there or how useful it was compared to the various README's, but presumably it provided additional information

Re: [Wireshark-dev] [Wireshark-commits] master 7ced085: emem is dead! Long live wmem!

2015-02-04 Thread Jeff Morriss
It wasn't an extremely interesting page (IMO)... It mostly described the idea behind the project and its (undoubtedly waaay out of date) status. On 02/04/15 10:49, Maynard, Chris wrote: Might that be a little hasty to remove the EMEM page so soon? I'm sure there are developers still working

Re: [Wireshark-dev] [Wireshark-commits] master 7ced085: emem is dead! Long live wmem!

2015-02-04 Thread Evan Huus
Woohoo! On Wed, Feb 4, 2015 at 2:00 AM, Wireshark code review code-review-do-not-re...@wireshark.org wrote: URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=7ced085550d030ea10525d650c8d5d8dc7c99684 Submitter: Anders Broman (a.broma...@gmail.com) Changed: branch:

Re: [Wireshark-dev] Undissected packet bytes

2015-02-04 Thread Evan Huus
I'm not sure why it would work differently in the two places if those print statements are the same. In general I would expect the requirements for that method to be: - *after* full dissection is finished - dissection must have been run with the full tree enabled (as when selecting a packet in the

Re: [Wireshark-dev] [Wireshark-commits] master 7ced085: emem is dead! Long live wmem!

2015-02-04 Thread Alexis La Goutte
On Wed, Feb 4, 2015 at 4:04 PM, Evan Huus eapa...@gmail.com wrote: Woohoo! +1 :-) Big Thanks Michael ! On Wed, Feb 4, 2015 at 2:00 AM, Wireshark code review code-review-do-not-re...@wireshark.org wrote: URL:

Re: [Wireshark-dev] [Wireshark-commits] master 7ced085: emem is dead! Long live wmem!

2015-02-04 Thread Jeff Morriss
So I deleted the EMEMification page from the wiki but I'm not sure what to do with the Canary page: http://wiki.wireshark.org/Development/Canary It looks like wmem does use canaries (under some circumstances?) so maybe it shouldn't be deleted. But I'm not familiar enough with what wmem is