Re: [Wireshark-dev] [Wireshark-commits] master 64cd827: Have dissect_usb_setup_request handle non-standard requests

2014-08-26 Thread Pascal Quantin
Le 27 août 2014 02:25, "Stalley, Sean" a écrit : > > Martin, > > I was able to set up a notification via the web interface. > > Under "Username"->Settings->Watched Projects > Set project to: wireshark > and the conditional to: file:"epan/dissectors/packet-usb.c" > > I think we should discuss how t

Re: [Wireshark-dev] [Wireshark-commits] master 64cd827: Have dissect_usb_setup_request handle non-standard requests

2014-08-26 Thread Stalley, Sean
Martin, I was able to set up a notification via the web interface. Under "Username"->Settings->Watched Projects Set project to: wireshark and the conditional to: file:"epan/dissectors/packet-usb.c" I think we should discuss how to clean up the usb dissector. It seems like we are running into th

[Wireshark-dev] Fwd: [Wireshark-bugs] [Bug 10410] Buildbot crash output: fuzz-2014-08-25-8573.pcap

2014-08-26 Thread Evan Huus
Gerald, dunno if this is possible or not, but the Gerrit integration probably shouldn't be marking bugs as "FIXED" when they've already been resolved into some other state... Don't think it's a big deal though, just seems odd. Evan -- Forwarded message -- From: Date: Tue, Aug 26

Re: [Wireshark-dev] should extcap dir be a preference?

2014-08-26 Thread Guy Harris
On Aug 26, 2014, at 6:57 AM, Martin Kaiser wrote: > at the moment, the directory for the extcap binaries is defined at > compile time. > > When I first tried extcap a year ago, I created a patch to make the > extcap directory a preference. This was much easier for me to handle > when I played w

Re: [Wireshark-dev] should extcap dir be a preference?

2014-08-26 Thread Alexis La Goutte
On Tue, Aug 26, 2014 at 4:31 PM, Pascal Quantin wrote: > M > Le 26 août 2014 15:59, "Martin Kaiser" a écrit : > > >> >> Hi, >> >> at the moment, the directory for the extcap binaries is defined at >> compile time. >> >> When I first tried extcap a year ago, I created a patch to make the >> extcap

Re: [Wireshark-dev] should extcap dir be a preference?

2014-08-26 Thread Pascal Quantin
Le 26 août 2014 20:00, "Alexis La Goutte" a écrit : > > On Tue, Aug 26, 2014 at 4:31 PM, Pascal Quantin > wrote: > > M > > Le 26 août 2014 15:59, "Martin Kaiser" a écrit : > > > > > >> > >> Hi, > >> > >> at the moment, the directory for the extcap binaries is defined at > >> compile time. > >> >

Re: [Wireshark-dev] [Wireshark-commits] master 64cd827: Have dissect_usb_setup_request handle non-standard requests

2014-08-26 Thread Alexis La Goutte
On Tue, Aug 26, 2014 at 12:14 PM, Graham Bloice wrote: > On 26 August 2014 11:05, Martin Kaiser wrote: >> >> >> Is there a way to "subscribe" to gerrit change requests so that I'm >> notified when a new USB-related change is uploaded? >> > > I have no idea if this will do what you want, but there

Re: [Wireshark-dev] Equivalency between APIs

2014-08-26 Thread mmann78
Yes it's true assuming "ti_channel" isn't otherwise used. -Original Message- From: Juan Jose Martin Carrascosa To: Developer support list for Wireshark Sent: Tue, Aug 26, 2014 11:32 am Subject: Re: [Wireshark-dev] Equivalency between APIs Thanks for your reply. Regarding the last

Re: [Wireshark-dev] Equivalency between APIs

2014-08-26 Thread Juan Jose Martin Carrascosa
Thanks for your reply. Regarding the last comment, just to double check since I am not very experienced on this... it would be like: channel_tree = proto_tree_add_subtree_format(rtps_parameter_tree, tvb, off, 0, ett_rtps_locator_filter_channel, NULL, "Channel[%u]", ch); True? - Juanj

Re: [Wireshark-dev] Equivalency between APIs

2014-08-26 Thread mmann78
You are correct. proto_tree_add_subtree is for no printf-style arguments, proto_tree_add_subtree_format is for when you need printf-style arguments for your subtree. One thing to check (if you're compiler doesn't do it for you because somebody's will) is that you may no longer need the proto

Re: [Wireshark-dev] should extcap dir be a preference?

2014-08-26 Thread Pascal Quantin
M Le 26 août 2014 15:59, "Martin Kaiser" a écrit : > > Hi, > > at the moment, the directory for the extcap binaries is defined at > compile time. > > When I first tried extcap a year ago, I created a patch to make the > extcap directory a preference. This was much easier for me to handle > when I

[Wireshark-dev] Equivalency between APIs

2014-08-26 Thread Juan Jose Martin Carrascosa
Hi all, I need to remove the proto_tree_add_text calls, and I wanted to know if this is equivalent: Before: ti_channel = proto_tree_add_text(rtps_parameter_tree, tvb, off, 0, "Channel[%u]", ch); channel_tree = proto_item_add_subtree(ti_channel, ett_rtps_locator_filter_channel);

[Wireshark-dev] should extcap dir be a preference?

2014-08-26 Thread Martin Kaiser
Hi, at the moment, the directory for the extcap binaries is defined at compile time. When I first tried extcap a year ago, I created a patch to make the extcap directory a preference. This was much easier for me to handle when I played with several extcaps that needed frequent recompiling. I'd l

Re: [Wireshark-dev] [Wireshark-commits] master 64cd827: Have dissect_usb_setup_request handle non-standard requests

2014-08-26 Thread Martin Kaiser
Hi Pascal, Thus wrote Pascal Quantin (pascal.quan...@gmail.com): > I spotted a regression myself (that seems similar to what you report) and > proposed https://code.wireshark.org/review/#/c/3858/ to fix it. Could you > give it a try on your sample? looks good to me, I just merged it. Best regar

Re: [Wireshark-dev] [Wireshark-commits] master 64cd827: Have dissect_usb_setup_request handle non-standard requests

2014-08-26 Thread Graham Bloice
On 26 August 2014 11:05, Martin Kaiser wrote: > > Is there a way to "subscribe" to gerrit change requests so that I'm > notified when a new USB-related change is uploaded? > > I have no idea if this will do what you want, but there does appear to be email notification from Gerrit: https://code.wi

Re: [Wireshark-dev] [Wireshark-commits] master 64cd827: Have dissect_usb_setup_request handle non-standard requests

2014-08-26 Thread Pascal Quantin
Hi Martin, 2014-08-26 12:05 GMT+02:00 Martin Kaiser : > > The new req_type is the same as the existing type. There's a > switch(type) {...} followed by in if (req_type==...). > > My goal is to have only one call to try_dissect_next_protocol(). In > dissect_usb_common(), we dissect the standard fi

Re: [Wireshark-dev] [Wireshark-commits] master 64cd827: Have dissect_usb_setup_request handle non-standard requests

2014-08-26 Thread Martin Kaiser
The new req_type is the same as the existing type. There's a switch(type) {...} followed by in if (req_type==...). My goal is to have only one call to try_dissect_next_protocol(). In dissect_usb_common(), we dissect the standard fields in the main switch-statement and call try_dissect_next_protoc

Re: [Wireshark-dev] Gerrit patches with trailing whitespace

2014-08-26 Thread Peter Kjellerstedt
> -Original Message- > From: wireshark-dev-boun...@wireshark.org [mailto:wireshark-dev- > boun...@wireshark.org] On Behalf Of Evan Huus > Sent: den 25 augusti 2014 19:32 > To: Developer support list for Wireshark > Subject: Re: [Wireshark-dev] Gerrit patches with trailing whitespace > > On

[Wireshark-dev] newbie git review -s issue

2014-08-26 Thread David Arnold
Hi all, I attempted to get my wireshark dev env updated to git today, and ran into an issue with the 'git review -s' part of the instructions. The error message was: Problems encountered installing commit-msg hook The following command failed with exit code 104 "GET https://d...@code.wiresh