Ethereal-dev: Re: [Ethereal-dev] Patch (PDML fields less than one byte)
Note: This archive is from the project's previous web site, ethereal.com. This list is no longer active.
From: Gilbert Ramirez <gilbertr@xxxxxxxxx>
Date: Fri, 27 May 2005 10:00:20 -0500
Here is what I have found: cops.in-int.ipv4 has a bitmask 0xffff but is type FT_IPv4 cops.in-int.ipv6 has a bitmask 0xffff but is type FT_IPv6 cops.out-int.ipv4 has a bitmask 0xffff but is type FT_IPv4 cops.out-int.ipv6 has a bitmask 0xffff but is type FT_IPv6 cops.pdprediraddr.ipv4 has a bitmask 0xffff but is type FT_IPv4 cops.pdprediraddr.ipv6 has a bitmask 0xffff but is type FT_IPv6 cops.lastpdpaddr.ipv4 has a bitmask 0xffff but is type FT_IPv4 cops.lastpdpaddr.ipv6 has a bitmask 0xffff but is type FT_IPv6 cops.pc_mm_classifier_src_addr has a bitmask 0xffff but is type FT_IPv4 cops.pc_mm_classifier_dst_addr has a bitmask 0xffff but is type FT_IPv4 image-gif.image has a bitmask 0x80 but is type FT_NONE I wrote a script to find this information; I'm checking in that script now. Gerald - the script could be added to buildbot: $ ./tools/ftsanity.py location-of-tethereal --gilbert On 5/27/05, vasanth.manickam@xxxxxx <vasanth.manickam@xxxxxx> wrote: > Hi, > Can you send me some more examples... I can help you clean them up.. > Regards > Vasanth Manickam > > > -----Original Message----- > From: ethereal-dev-bounces@xxxxxxxxxxxx > [mailto:ethereal-dev-bounces@xxxxxxxxxxxx] On Behalf Of Gilbert Ramirez > Sent: 26 May 2005 16:19 > To: Ethereal development > Subject: Re: [Ethereal-dev] Patch (PDML fields less than one byte) > > > Thanks. I'm testing this; it turns out there are some non-integer fields > with bitmasks defined for them, like FT_NONE fields! I'm tracking these > down. > > --gilbert > > On 5/25/05, vasanth.manickam@xxxxxx <vasanth.manickam@xxxxxx> wrote: > > Sorry for spamming again.. I am unable to format the patch in the > > mail.. Attaching the file now... > > > > > > -----Original Message----- > > From: ethereal-dev-bounces@xxxxxxxxxxxx > > [mailto:ethereal-dev-bounces@xxxxxxxxxxxx] On Behalf Of > > vasanth.manickam@xxxxxx > > Sent: 25 May 2005 16:31 > > To: ethereal-dev@xxxxxxxxxxxx; gram@xxxxxxxxxxxxxxx > > Subject: RE: [Ethereal-dev] Patch (PDML fields less than one byte) > > > > > > Oops in the previous mail the patch seems to be formatted all wrong... > > > > --- print1.c 2005-05-25 13:13:53.000000000 +0100 > > +++ print.c 2005-05-25 16:09:25.301581248 +0100 > > @@ -346,6 +346,13 @@ > > } > > if (fi->length > 0) { > > fputs("\" value=\"", pdata->fh); > > + > > + if(fi->hfinfo->bitmask!=0){ > > + > > fprintf(pdata->fh,"%X",fvalue_get_integer(&fi->value)); > > + fputs("\" > > unmaskedvalue=\"",pdata->fh); > > + > > write_pdml_field_hex_value(pdata,fi); > > + } > > + else > > write_pdml_field_hex_value(pdata,fi); > > } > > } > > > > > > -----Original Message----- > > From: ethereal-dev-bounces@xxxxxxxxxxxx > > [mailto:ethereal-dev-bounces@xxxxxxxxxxxx] On Behalf Of > > vasanth.manickam@xxxxxx > > Sent: 25 May 2005 16:21 > > To: gram@xxxxxxxxxxxxxxx; ethereal-dev@xxxxxxxxxxxx > > Subject: RE: [Ethereal-dev] Patch (PDML fields less than one byte) > > > > > > Ok I have fixed the patch to accommodate "unmaskedvalue" and also > > corrected a small error which applied the changes to some fields which > > > were not necessarily subfields. I am new to contributing code to open > > > source projects... Please guide me on how I can get this patch > > verified and make it into the main build... > > > > Vasanth Manickam > > > > > > --- print.c 2005-05-25 13:13:53.000000000 +0100 > > +++ print1.c 2005-05-25 16:09:25.301581248 +0100 > > @@ -346,6 +346,13 @@ > > } > > if (fi->length > 0) { > > fputs("\" value=\"", pdata->fh); > > + > > + if(fi->hfinfo->bitmask!=0){ > > + fprintf(pdata->fh, "%X", > > fvalue_get_integer(&fi->value)); > > + fputs("\" unmaskedvalue=\"", > > pdata->fh); > > + > > write_pdml_field_hex_value(pdata, fi); > > + } > > + else > > write_pdml_field_hex_value(pdata, fi); > > } > > } > > > > > > > > > > -----Original Message----- > > From: ethereal-dev-bounces@xxxxxxxxxxxx > > [mailto:ethereal-dev-bounces@xxxxxxxxxxxx] On Behalf Of Gilbert > > Ramirez > > Sent: 24 May 2005 15:38 > > To: Ethereal development > > Subject: Re: [Ethereal-dev] Patch (PDML fields less than one byte) > > > > > > Upon further reading of the PDML specification at: > > > > http://analyzer.polito.it/30alpha/docs/dissectors/PDMLSpec.htm > > > > I see that you're right, but if we're going to show bit values in hex > > in the "value" attribute, then we need to also add the "unmaskedvalue" > > > attribute for those fields. The "unmaskedvalue" attribute will show > > the hex value of the entire byte or bytes. > > > > --gilbert > > > > On 5/24/05, vasanth.manickam@xxxxxx <vasanth.manickam@xxxxxx> wrote: > > > > > > The export PDML features creates an xml file where the value of the > > > subfields less than 1 byte have the wrong value. Instead of just the > > > > bit or part of the byte the value field contains the entire byte. > > > > > > Example:- > > > > > > <field name="tcp.flags.cwr" showname="0... .... = Congestion Window > > > Reduced > > > (CWR): Not set" size="1" pos="47" show="0" value="18" /> > > > > > > > > > The value field is supposed to represent only the 1st bit but the > > > entire byte is represented as value 18. This caused a lot of problem > > > > in applications which process the xml file for further use. > > > > > > The below patch seems to solve this problem by modifying the value > > > field that is written to the xml file > > > > > > > > > ------------------------------------------------- > > > > > > --- print.c 2005-05-24 12:52:55.000000000 +0100 > > > +++ print_new.c 2005-05-24 13:08:56.593591576 +0100 > > > @@ -346,6 +346,10 @@ > > > } > > > if (fi->length > 0) { > > > fputs("\" value=\"", pdata->fh); > > > + if (!fi->rep) > > > + print_escaped_xml(pdata->fh, &dfilter_string[chop_len]); > > > + else > > > write_pdml_field_hex_value(pdata, fi); > > > } > > > } > > > _______________________________________________ > > > Ethereal-dev mailing list > > > Ethereal-dev@xxxxxxxxxxxx > > > http://www.ethereal.com/mailman/listinfo/ethereal-dev > > > > > > > > > > > > > _______________________________________________ > > Ethereal-dev mailing list > > Ethereal-dev@xxxxxxxxxxxx > > http://www.ethereal.com/mailman/listinfo/ethereal-dev > > > > _______________________________________________ > > Ethereal-dev mailing list > > Ethereal-dev@xxxxxxxxxxxx > > http://www.ethereal.com/mailman/listinfo/ethereal-dev > > > > _______________________________________________ > > Ethereal-dev mailing list > > Ethereal-dev@xxxxxxxxxxxx > > http://www.ethereal.com/mailman/listinfo/ethereal-dev > > > > > > _______________________________________________ > > Ethereal-dev mailing list > > Ethereal-dev@xxxxxxxxxxxx > > http://www.ethereal.com/mailman/listinfo/ethereal-dev > > > > > > > > > > _______________________________________________ > Ethereal-dev mailing list > Ethereal-dev@xxxxxxxxxxxx > http://www.ethereal.com/mailman/listinfo/ethereal-dev > >
- References:
- RE: [Ethereal-dev] Patch (PDML fields less than one byte)
- From: vasanth.manickam
- RE: [Ethereal-dev] Patch (PDML fields less than one byte)
- Prev by Date: Re: [Ethereal-dev] (no subject)
- Next by Date: [Ethereal-dev] Dissector for synergy
- Previous by thread: RE: [Ethereal-dev] Patch (PDML fields less than one byte)
- Next by thread: [Ethereal-dev] Problems with capture_if_details_dlg.c SVN-14417
- Index(es):