Wireshark mailing list archives
Re: Wireshark PIDL generated dissectors
From: ronnie sahlberg <ronniesahlberg () gmail com>
Date: Thu, 3 Oct 2013 19:44:29 -0700
Cool. And we contact you when samba-PIDL no longer can generate compileable wireshark dissectors? On Thu, Oct 3, 2013 at 7:13 PM, Andrew Bartlett <abartlet () samba org> wrote:
On Tue, 2013-10-01 at 10:55 -0400, mmann78 () netscape net wrote:The check_col function in the Wireshark source has been deprecated for awhile, but never officially removed. I've been modifying the source (removing calls) so that it can be officially removed. The last big hurdle was the DCE/RPC generated dissector files. I've had a difficult time trying to regenerate the dissectors with the necessary modified source, so I just manually updated dissectors to mimic what the source should generate. Attached is a patch that contains the modifications to "PIDL source" that generates the dissector files (and not the dissector files themselves). I couldn't tell from the Wireshark wiki [1] if this was maintained by samba or Wireshark, so now both will now have a copy, with the source already committed to Wireshark repository [3]. Patch is also available in Wireshark bugzilla [3]. Michael Mann [1] http://wiki.wireshark.org/Pidl [2] http://anonsvn.wireshark.org/viewvc?view=revision&revision=52313 [3] https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8804PIDL is maintained by the Samba team, the canonical repository is in our master branch of git://git.samba.org/samba.git From the Samba perspective: I would warn that patching the generated code only passes the pain of fixing this properly down to the next person who needs to work here. As improvements are made to the IDL files or the parser, the generated files need to be re-generated, and then someone has the un-enviable choice of throwing away manual changes, or reverse engineering these back into the compiler. Even if you think you have modified them correctly to match your PIDL changes, please persevere to get the files re-generated using PIDL. Only then will we really know the generator generates the code correctly, and only then will the solution be long-term maintainable. Thanks, Andrew Bartlett -- Andrew Bartlett http://samba.org/~abartlet/ Authentication Developer, Samba Team http://samba.org Samba Developer, Catalyst IT http://catalyst.net.nz
___________________________________________________________________________ Sent via: Wireshark-dev mailing list <wireshark-dev () wireshark org> Archives: http://www.wireshark.org/lists/wireshark-dev Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev mailto:wireshark-dev-request () wireshark org?subject=unsubscribe
Current thread:
- Wireshark PIDL generated dissectors mmann78 (Oct 01)
- Re: Wireshark PIDL generated dissectors ronnie sahlberg (Oct 03)
- Re: Wireshark PIDL generated dissectors Andrew Bartlett (Oct 05)
- Re: Wireshark PIDL generated dissectors ronnie sahlberg (Oct 03)
- Re: Wireshark PIDL generated dissectors Andrew Bartlett (Oct 05)
- Re: Wireshark PIDL generated dissectors ronnie sahlberg (Oct 03)
- Re: Wireshark PIDL generated dissectors Matthieu Patou (Oct 06)
- Re: Wireshark PIDL generated dissectors Evan Huus (Oct 06)
- Re: Wireshark PIDL generated dissectors Joerg Mayer (Oct 06)
- Re: Wireshark PIDL generated dissectors Matthieu Patou (Oct 07)
- Re: Wireshark PIDL generated dissectors Jeff Morriss (Oct 07)
- Re: Wireshark PIDL generated dissectors Joerg Mayer (Oct 07)
- Re: Wireshark PIDL generated dissectors Anders Broman (Oct 07)
- Re: Wireshark PIDL generated dissectors Matthieu Patou (Oct 08)
- Re: Wireshark PIDL generated dissectors ronnie sahlberg (Oct 03)