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

2012-01-23 Thread Tony Trinh
On Mon, Jan 23, 2012 at 4:56 PM, Maynard, Chris < christopher.mayn...@gtech.com> wrote: > > > From: wireshark-dev-boun...@wireshark.org [mailto: > wireshark-dev-boun...@wireshark.org] On Behalf Of Tony Trinh > > Sent: Monday, January 23, 2012 4:46 PM > > To: Developer support list for Wireshark >

Re: [Wireshark-dev] warning in packet-rohc, r40673

2012-01-23 Thread Jeff Morriss
Martin Kaiser wrote: Hi Anders and all, I get a warning (treated as an error) martin@greta:~/src/wireshark.svn$ svn info ... Last Changed Author: etxrab Last Changed Rev: 40673 Last Changed Date: 2012-01-23 22:13:10 +0100 (Mon, 23 Jan 2012) martin@greta:~/src/wireshark.svn$ uname -a Linux gret

[Wireshark-dev] warning in packet-rohc, r40673

2012-01-23 Thread Martin Kaiser
Hi Anders and all, I get a warning (treated as an error) martin@greta:~/src/wireshark.svn$ svn info ... Last Changed Author: etxrab Last Changed Rev: 40673 Last Changed Date: 2012-01-23 22:13:10 +0100 (Mon, 23 Jan 2012) martin@greta:~/src/wireshark.svn$ uname -a Linux greta 2.6.32-5-amd64 #1

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

2012-01-23 Thread Maynard, Chris
> From: wireshark-dev-boun...@wireshark.org > [mailto:wireshark-dev-boun...@wireshark.org] On Behalf Of Tony Trinh > Sent: Monday, January 23, 2012 4:46 PM > To: Developer support list for Wireshark > Subject: Re: [Wireshark-dev] [Wireshark-commits] rev 40644: > /trunk/epan/dissectors/ /trunk/ep

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

2012-01-23 Thread Tony Trinh
On Mon, Jan 23, 2012 at 4:12 PM, Maynard, Chris < christopher.mayn...@gtech.com> wrote: > > I guess I can use notepad++ from now on for this type of editing, but if > you've got any ideas as to how to get it work under cygwin, that would be > appreciated. > Thanks. > - Chris > > I think I know wha

Re: [Wireshark-dev] pcapng, must opt_commentstring be?0-terminated?

2012-01-23 Thread Chris Maynard
Martin Kaiser writes: > I was wondering why we need a static buffer at all. It looks like the > intention is to keep using the same buffer for each option that we > parse. When reading an option, how about checking the length first and > then allocating the buffer dynamically? We could then remov

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

2012-01-23 Thread Maynard, Chris
> -Original Message- > From: wireshark-dev-boun...@wireshark.org [mailto:wireshark-dev- > boun...@wireshark.org] On Behalf Of Guy Harris > Sent: Monday, January 23, 2012 3:24 PM > To: Developer support list for Wireshark > Subject: Re: [Wireshark-dev] [Wireshark-commits] rev 40644: > /trunk

[Wireshark-dev] Ubuntu buildbot stuck

2012-01-23 Thread Jeff Morriss
Hi Gerald, Looks like the Ubuntu buildbot needs to be "kicked" again. -J ___ Sent via:Wireshark-dev mailing list Archives:http://www.wireshark.org/lists/wireshark-dev Unsubscribe: https://wireshark.org/mailman/optio

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

2012-01-23 Thread Tony Trinh
On Mon, Jan 23, 2012 at 3:08 PM, Maynard, Chris < christopher.mayn...@gtech.com> wrote: > > Now it displays as follows for me: > ? > > In Windows 7, I can copy paste *Роман Донченко* into plain-old Notepad, and it shows correctly to my surprise. Interestingly, if I copy that

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

2012-01-23 Thread Guy Harris
On Jan 23, 2012, at 12:08 PM, Maynard, Chris wrote: > Now it displays as follows for me: > ? What software is displaying it? At least in Mac OS X 10.6.8 when running commands in Terminal, both "more" (which, in honor of Ludwig Mies van der Rohe, is really just another li

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

2012-01-23 Thread Maynard, Chris
> -Original Message- > From: wireshark-dev-boun...@wireshark.org [mailto:wireshark-dev- > boun...@wireshark.org] On Behalf Of Gerald Combs > Sent: Monday, January 23, 2012 2:23 PM > To: Developer support list for Wireshark > Subject: Re: [Wireshark-dev] [Wireshark-commits] rev 40644: > /tru

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

2012-01-23 Thread Gerald Combs
On 1/23/12 9:34 AM, Jeff Morriss wrote: > Maynard, Chris wrote: >> >> I tried that but it didn't work for me. Are you able to? > > Yes, but it depends on the application being pasted into. For example, > I can copy DXDragon's name out of Firefox and into Thunderbird just fine > but I can't paste

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

2012-01-23 Thread Jeff Morriss
Maynard, Chris wrote: -Original Message- From: wireshark-dev-boun...@wireshark.org [mailto:wireshark-dev- boun...@wireshark.org] On Behalf Of Jeff Morriss Sent: Monday, January 23, 2012 11:55 AM To: wireshark-dev@wireshark.org Subject: Re: [Wireshark-dev] [Wireshark-commits] rev 40644: /t

Re: [Wireshark-dev] How do display filters work internally?

2012-01-23 Thread Chris Maynard
Maynard, Chris writes: > an "ip.addr" filter and it matched IP addresses in the IP protocol, but also with some bootp traffic. Apologies - the "ip.addr" filter works for the IP addresses in the IP header, but not for the bootp IP address fields, so there's still something else going wrong here t

Re: [Wireshark-dev] How do display filters work internally?

2012-01-23 Thread Maynard, Chris
> -Original Message- > From: wireshark-dev-boun...@wireshark.org [mailto:wireshark-dev- > boun...@wireshark.org] On Behalf Of Joerg Mayer > Sent: Thursday, January 19, 2012 5:41 PM > To: wireshark-dev@wireshark.org > Subject: [Wireshark-dev] How do display filters work internally? > > Hel

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

2012-01-23 Thread Maynard, Chris
> -Original Message- > From: wireshark-dev-boun...@wireshark.org [mailto:wireshark-dev- > boun...@wireshark.org] On Behalf Of Jeff Morriss > Sent: Monday, January 23, 2012 11:55 AM > To: wireshark-dev@wireshark.org > Subject: Re: [Wireshark-dev] [Wireshark-commits] rev 40644: > /trunk/epan/

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

2012-01-23 Thread Jeff Morriss
cmayn...@wireshark.org wrote: http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=rev&revision=40644 User: cmaynard Date: 2012/01/21 05:59 PM Log: Patch from DXDragon [AT] yandex.ru (using e-mail address instead of name because I don't know how to enter Cyrillic letters) to fix https://bugs.

Re: [Wireshark-dev] Iteration in dissectors?

2012-01-23 Thread Tyson Key
I eventually managed to get thing working by using a combination of a modified version of Chris's second approach, and use of tvb_new_subset_remaining() to capture data after the Number of Blocks byte. Since I've also managed to get Polling Request/Response, and Read Without Encryption Response pa