[Wireshark-dev] Wireshark PIDL generated dissectors

Pascal Quantin pascal.quantin at gmail.com
Sun Nov 3 12:37:29 MST 2013


Hi,

there is also a small issue spotted by checkAPIs.pl script (found in tools
folder) leading to a failure of the buildbot:
Error: the name for
hf_frsrpc_frsrpc_FrsVerifyPromotionParent___ndr_guid_size
("frsrpc.frsrpc_FrsVerifyPromotionParent.__ndr_guid_size") has leading
space in packet-dcerpc-frsrpc.c
Error: the name for
hf_frsrpc_frsrpc_FrsStartPromotionParent___ndr_guid_size
("frsrpc.frsrpc_FrsStartPromotionParent.__ndr_guid_size") has leading space
in packet-dcerpc-frsrpc.c
I fixed them manually in r53072 but you probably want to change the code
generator so as to remove the prepended spaces.

Best regards,
Pascal.

PS: checkhf.pl script also found the following error:
Unused entry: epan/dissectors/packet-dcerpc-frsrpc.c,
hf_frsrpc_frsrpc_CommPktChangeOrderCommand_file_name

2013/11/3 <mmann78 at netscape.net>

>  Matthieu,
>
> I committed
> 0010-frsrpc-Regenerate-frsrpc-due-to-changes-in-the-pidl-.patch and
> 0016-Regenerate-the-dnserver.patch with the necessary changes to get it to
> compile on Windows to r53067.  As the commit message mentions, I think the
> PIDL generator may need to be updated in order for dissectors to compile on
> Windows.
>
> Michael
>  -----Original Message-----
> From: Matthieu Patou <mat at samba.org>
> To: mmann78 <mmann78 at netscape.net>; wireshark-dev <
> wireshark-dev at wireshark.org>
> Cc: abartlet <abartlet at samba.org>; samba-technical <
> samba-technical at samba.org>
>  Sent: Sun, Oct 27, 2013 1:41 pm
> Subject: Re: [Wireshark-dev] Wireshark PIDL generated dissectors
>
>  On 10/27/2013 09:16 AM, mmann78 at netscape.net wrote:
>
> I may have slightly misspoke about the patch in bug 9301 *causing* the
> compile errors, but the patch in bug 9301 had the *same* compile errors
> on Windows as the generated dissectors.  I originally committed the patch
> in r52734 before realizing the compiler errors which I corrected in r52735
> (with a simple cast).
>
> Yes I saw your patch for the bug 9301. But in theory I could resubmit the
> two patches because the errors weren't in the patch but in the use it made
> of the funcion in bug 9301
>
> I don't have access to a Linux box, but I suspect if you used Linux to
> generate the patch in bug 9301, you won't see this issue either with the
> generated dissectors. It appears the generated dissectors need extra
> casting to compile on Windows and I didn't just want to make the fix in the
> Wireshark source.
>
> Also do you have some kind of buildfarm, I'm ok to build patches there
> before submissions.
>
> Matthieu.
>
>  -----Original Message-----
> From: Matthieu Patou <mat at samba.org>
> To: mmann78 <mmann78 at netscape.net>; wireshark-dev
> <wireshark-dev at wireshark.org>
> Cc: abartlet <abartlet at samba.org>; samba-technical
> <samba-technical at samba.org>
> Sent: Sat, Oct 26, 2013 7:26 pm
> Subject: Re: [Wireshark-dev] Wireshark PIDL generated dissectors
>
>  On 10/21/2013 12:48 PM, mmann78 at netscape.net wrote:
> >
> > Checked most of the patches into r52744 (http://anonsvn.wireshark.org/viewvc?view=revision&revision=52744)
> >
> > Didn't integrate
> > 0010-frsrpc-Regenerate-frsrpc-due-to-changes-in-the-pidl-.patch
> > 0016-Regenerate-the-dnserver.patch
> >
> > due to compile errors on Windows from applying the patch in bug 9301
> (https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9301).
> >
> > See r52735 (http://anonsvn.wireshark.org/viewvc?view=revision&revision=52735)
> for how it was fixed.  Not sure if the PIDL compiler has to be changed for this
> one.
> Can you tell me what were the compile errors on linux, it's seems just
> fine with just this patch on top of the recent svn trunk.
>
> I just have warnings for dnsserver:
> /home/mat/workspace/wireshark/epan/dissectors/packet-dcerpc-dnsserver.c:
> In function ‘dnsserver_dissect_element_DNS_RPC_RECORD_Type’:
> /home/mat/workspace/wireshark/epan/dissectors/packet-dcerpc-dnsserver.c:1381:2:
> warning: passing argument 7 of ‘dnsserver_dissect_enum_DNS_RECORD_TYPE’
> from incompatible pointer type [enabled by default]
> /home/mat/workspace/wireshark/epan/dissectors/packet-dcerpc-dnsserver.c:1008:1:
> note: expected ‘guint32 *’ but argument is of type ‘guint16 *’
> /home/mat/workspace/wireshark/epan/dissectors/packet-dcerpc-dnsserver.c:
> At top level:
> /home/mat/workspace/wireshark/epan/dissectors/packet-dcerpc-dnsserver.c:1261:1:
> warning: ‘dnsserver_dissect_element_DNS_RPC_NAME_NameLength’ defined but
> not used [-Wunused-function]
> /home/mat/workspace/wireshark/epan/dissectors/packet-dcerpc-dnsserver.c:1269:1:
> warning: ‘dnsserver_dissect_element_DNS_RPC_NAME_Name’ defined but not
> used [-Wunused-function]
> /home/mat/workspace/wireshark/epan/dissectors/packet-dcerpc-dnsserver.c:1276:1:
> warning: ‘dnsserver_dissect_element_DNS_RPC_NAME_Name_’ defined but not
> used [-Wunused-function]
>
> Matthieu
>
>
>
> --
> Matthieu Patou
> Samba Teamhttp://samba.org
>
>
> ___________________________________________________________________________
> Sent via:    Wireshark-dev mailing list <wireshark-dev at wireshark.org>
> Archives:    http://www.wireshark.org/lists/wireshark-dev
> Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
>              mailto:wireshark-dev-request at wireshark.org
> ?subject=unsubscribe
>


More information about the samba-technical mailing list