Wireshark · Ethereal-dev: SV: SV: [Ethereal-dev] iax2 dissector patch

["Anders Broman" <a.broman@xxxxxxxxx>]

Hi,
The problem seems to be the line:
  memset(hf_iax2_ies,0xff,sizeof(hf_iax2_ies));

Removing the line makes iax2 compile, I'm not sure what the solution is
perhaps to do it as in packet-sip instead?
static gint hf_header_array[] = {
                -1, /* 0"Unknown-header" - Pad so that the real headers
start at index 1 */
:

Brg
Anders

-----Ursprungligt meddelande-----
Från: ethereal-dev-bounces@xxxxxxxxxxxx
[mailto:ethereal-dev-bounces@xxxxxxxxxxxx] För Richard van der Hoff
Skickat: den 2 augusti 2005 21:57
Till: Ethereal development
Ämne: Re: SV: [Ethereal-dev] iax2 dissector patch

Hi Anders,

Thanks for looking at this.

Anders Broman wrote:
> Hi,
> I get the following errors trying to apply the patch:
> register.c
> packet-iax2.c
> packet-iax2.c(596) : error C2275: 'circuit_t' : illegal use of this type
as
> an e
> xpression
>         ../..\epan/circuit.h(48) : see declaration of 'circuit_t'
> packet-iax2.c(596) : error C2065: 'res' : undeclared identifier
> packet-iax2.c(600) : warning C4047: 'function' : 'struct circuit *'
differs
> in l
> evels of indirection from 'int '
> packet-iax2.c(600) : warning C4024: 'circuit_add_proto_data' : different
> types f
> or formal and actual parameter 1
> packet-iax2.c(607) : warning C4047: 'return' : 'struct circuit *' differs
in
> lev
> els of indirection from 'int '

Oops - sorry about these. gcc allows you to declare variables in the 
middle of a function, so I missed these. I've attached an updated patch.

> packet-iax2.c(1337) : warning C4013: 'get_timedelta' undefined; assuming
> extern
> returning int

Yes - as I said, the changes rely on a previous patch, which hasn't been 
checked in. I've attached that patch too: the explanation to go with it 
is in the post entitled "negative relative time fixes" on 5th July.

> packet-iax2.c(1732) : error C2143: syntax error : missing ';' before
'type'
> packet-iax2.c(2165) : fatal error C1013: compiler limit : too many open
> parenthe
> ses
> 
Hrm. I have no idea what these are about, I'm afraid. I don't suppose 
fixing the other errors makes them go away?

Richard

-- 
Richard van der Hoff <richardv@xxxxxxxxxxxxx>
Systems Analyst
Tel: +44 (0) 845 666 7778
http://www.mxtelecom.com