Re: [Wireshark-dev] Compilation failure - privileges.c: In function 'relinquish_special_privs_perm' - ignoring return value of 'setresgid', declared with attribute warn_unused_result [-Werror=unused-r

2012-08-21 Thread Guy Harris
On Aug 20, 2012, at 12:49 PM, Kaul wrote: If it were git, I'm sure I could easily use 'git bisect' and find the issue. As Evan Huus indicated, what you probably need to bisect is compiler updates. :-) The offending routine is probably relinquish_special_privs_perm() (which has failed to

Re: [Wireshark-dev] DeviceNet dissector

2012-08-21 Thread Hans-Jörgen Gunnarsson
I have now built a program that takes the frames and sends them into Wireshark through a pipe. Works really good, thanks for the tip. In Wireshark the frames go into the SocketCAN dissector which I have modified and added DeviceNet to. Next step is to build the DeviceNet Dissector. I want to

Re: [Wireshark-dev] DeviceNet dissector

2012-08-21 Thread mmann78
Unfortunately the CIP in DeviceNet is drastically different than the CIP that runs over Ethernet/IP. I think you'd be much better off just writing a pure DeviceNet dissector because I think you'd spend just as much time trying to manipulate packets (if its even possible) to make them look

Re: [Wireshark-dev] [Wireshark-commits] rev 44559: /trunk/epan/dissectors/ /trunk/epan/dissectors/: packet-sctp.c

2012-08-21 Thread Jeff Morriss
mman...@netscape.net wrote: I guess this makes #3 from http://www.wireshark.org/lists/wireshark-dev/201208/msg00164.html an unfortunate yes. I guess I had the same idea you did - slowly Yeah I noticed that after I sent the below email; I guess you can tell the order in which I read the

Re: [Wireshark-dev] [Wireshark-commits] rev 44559: /trunk/epan/dissectors/ /trunk/epan/dissectors/: packet-sctp.c

2012-08-21 Thread mmann78
SCTP isn't the only dissector doing it's own reassembly (at least from what I can see grepping g_ allocation calls), but I agree having it use the reassembly API (if possible) would be preferred over reverting my se_ alloc changes. Unfortunately, sometimes ignorance is the reason a provided

Re: [Wireshark-dev] [Wireshark-commits] rev 44559: /trunk/epan/dissectors/ /trunk/epan/dissectors/: packet-sctp.c

2012-08-21 Thread Michael Tuexen
On Aug 21, 2012, at 8:26 PM, Jeff Morriss wrote: mman...@netscape.net wrote: I guess this makes #3 from http://www.wireshark.org/lists/wireshark-dev/201208/msg00164.html an unfortunate yes. I guess I had the same idea you did - slowly Yeah I noticed that after I sent the below email; I

Re: [Wireshark-dev] Wireshark-commits: [Wireshark-commits] rev 44161: /trunk/epan/dissectors/ /trunk/epan/dissectors/: packet-gmr1_bcch.c packet-gmr1_common.c packet-gmr1_rr.c

2012-08-21 Thread Pascal Quantin
2012/8/20 mman...@netscape.net Sylvain, The checkdisplayfilter.pl script reflects my interpretation of the desired display filter format, and since there hasn't been that much feedback on the outputted results (with Pascal's comments on the GSM dissectors being the exception), I

Re: [Wireshark-dev] Wireshark-commits: [Wireshark-commits] rev 44161: /trunk/epan/dissectors/ /trunk/epan/dissectors/: packet-gmr1_bcch.c packet-gmr1_common.c packet-gmr1_rr.c

2012-08-21 Thread mmann78
I'll revert the GMR-1 display filter name changes and update checkdisplayfilter.pl to accept periods in the base filter name, but I will note their existance in another FILE LIST grouping in case other dissectors don't have a valuable reason to do so. -Original Message- From: Pascal