Wireshark-dev: Re: [Wireshark-dev] Add the next Tvb packet to current tvb before dissecting
From: wsgd <[email protected]>
Date: Thu, 13 Aug 2009 20:20:07 +0200
The warning is saying that the parameter 'pinfo' is NOT used inside the 
function.
It is not important.
Try this : "packet_info *pinfo _U_," instead of "packet_info *pinfo," (not sure that the warning will disappear).

The "storage" error ?
Absolutely not sure.
Try :
- remove the static keyword
- put the 2 functions in the same file where you call tcp_dissect_pdus
- put the 2 functions before the place where you call tcp_dissect_pdus
- do not declare function inside functions



Olivier


Clifford Sibanda a écrit :
 Thank you Olivier for responding

The reason why I attempted to do the reassemble the long way wa because using the tcp_dissect_pdus I get the following errors
:647:error: invalid storage class for function 'dissect_l2i_message'
for this line
static void dissect_l2i_message(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree)
{

and same error for this line
:652: error: invalid storage class for function 'get_l2i_message_len'
for this line
static guint get_l2i_message_len(packet_info *pinfo, tvbuff_t *tvb, int offset)
{

also
a warning

:651: warning: unused parameter 'pinfo'

for the same line
static guint get_l2i_message_len(packet_info *pinfo, tvbuff_t *tvb, int offset)
{

I'm not sure what it is that I'm doing wrong here

I have taken note of void assignment by next_tvb=
And i have removed it but I suppose if I can get the tcp_dissect_pdus to work for me the better because my protocol is running atop tcp.
Regards

Clifford
------------------------------------------------------------------------
*From:* wsgd <[email protected]>
*To:* Developer support list for Wireshark <[email protected]>
*Sent:* Wednesday, August 12, 2009 8:27:38 PM
*Subject:* Re: [Wireshark-dev] Add the next Tvb packet to current tvb before dissecting
Hello,

The displayed error says that tvb_set_child_real_data_tvbuff returns
void (nothing).
See epan/tvbuff.h
So you are trying : next_tvb=void


Other thing :
I do not understand why you are trying to do reassembly by yourself
since this is the goal of tcp_dissect_pdus.



Olivier



Clifford Sibanda a écrit :
>
> Good day
>
> I would like to get some assistance if anyone out there can.
>
> I am working a plugin dissector that handles data from an asynchronous
> source. The asynchronous source first sends a packet with just the
> message length of the next packet which contains the real data.
>
> What i would love to do is to keep this packet with just the length of
> the next packet and append to it the next packet before I proceed with
> dissecting.
>
> I have tried using the tcp_dissect_pdus and the longer reassembly
> method but I get this error using both methods:
>
> error: void value not ignored as it ought to be
>
> for this line
>
> next_tvb=tvb_set_child_real_data_tvbuff(tvb,next_tvb);
>
> I'm not sure what it is that I'm doing wrong
>
> And if there is a simpler way of just adding the next tvb without
> going the reassemble route I would like to use that one
>
> Thank you
>
> Clifford Sibanda
> Good day
>
>
> ------------------------------------------------------------------------
>
> ___________________________________________________________________________ > Sent via: Wireshark-dev mailing list <[email protected] <mailto:[email protected]>> > Archives: http://www.wireshark..org/lists/wireshark-dev <http://www.wireshark.org/lists/wireshark-dev>
> Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
> mailto:[email protected] <mailto:[email protected]>?subject=unsubscribe
___________________________________________________________________________
Sent via: Wireshark-dev mailing list <[email protected] <mailto:[email protected]>>
Archives:    http://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
mailto:[email protected] <mailto:[email protected]>?subject=unsubscribe
------------------------------------------------------------------------

___________________________________________________________________________
Sent via:    Wireshark-dev mailing list <[email protected]>
Archives:    http://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
             mailto:[email protected]?subject=unsubscribe