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@xxxxxxxxx> wrote:
> On Tue, 2013-10-01 at 10:55 -0400, mmann78@xxxxxxxxxxxx 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=8804
>
> PIDL 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
>
>