Wireshark-dev: Re: [Wireshark-dev] get_pdu_len signature change (TCP re-assembly) and SVN versi
sorry
but the old signature was wrong and had to be changed
there are unfortunately costs with living outside of the tree
On 11/21/06, Jacques, Olivier (PD&E IT Test) <olivier.jacques@xxxxxx> wrote:
Hello,
with recent Wireshark SVN versions, my TCP plugins which use re-assembly
are throwing exceptions.
I found that there is a change that occured in "get_pdu_len" signature
that happened in SVN 19751
(http://www.mail-archive.com/wireshark-commits@xxxxxxxxxxxxx/msg01181.ht
ml
<http://www.mail-archive.com/wireshark-commits@xxxxxxxxxxxxx/msg01181.ht
ml> ).
The corresponding documentation in README.developer should change too to
reflect that.
In 2.7.1 Using tcp_dissect_pdus(), the text should read:
-----------------------------------
a routine that takes as arguments a packet_info pointer, a tvbuff
pointer and an offset
value representing the offset into the tvbuff at which a PDU
begins and should return - *without* throwing an exception (it
is guaranteed that the number of bytes specified by the previous
argument to tcp_dissect_pdus is available, but more data might
not be available, so don't refer to any data past that) - the
total length of the PDU, in bytes;
-----------------------------------
I updated my plugins and it seems to be OK.
The ideal would be to protect the plugins from being incompatible from
one Wireshark version to another (I know binary compatibility is not
guaranteed, but I would like to make this as transparent as possible).
The first idea that comes to me is having access to the SVN release at
run time (compile time is not enough as we provide plugin binaries) and
test against this.
But maybe there is something that already exist for that?
Thanks,
Olivier.
