Ethereal-dev: Re: [Ethereal-dev] packet-smb.c:11327 - si->info_level = t2i->info_level

Note: This archive is from the project's previous web site, ethereal.com. This list is no longer active.

Date Index Thread Index Other Months All Mailing Lists
Date Prev Date Next Thread Prev Thread Next
From: "Ronnie Sahlberg" <sahlberg@xxxxxxxxxxxxxxxx>
Date: Wed, 20 Feb 2002 22:04:50 +1100
Hi list, Guy, Marcin.

This should make the request reply matching more correct since this patch
also checks the
cmd to see if they are compatible.

Marcin, can you try to apply this patch and see if it fixes your problem?



----- Original Message -----
From: "Guy Harris"
To: "Marcin Gryszkalis"
Cc: "ethereal-dev"
Sent: Tuesday, February 19, 2002 6:31 PM
Subject: Re: [Ethereal-dev] packet-smb.c:11327 - si->info_level =
t2i->info_level


> On Tue, Feb 19, 2002 at 08:26:10AM +0100, Marcin Gryszkalis wrote:
> > > What are the values of "si" and "t2i"?
> >
> > (gdb) print si
> > $4 = (smb_info_t *) 0xbfbfe79c
> > (gdb) print t2i
> > $5 = (smb_transact2_info_t *) 0x800a
> > (gdb) print *si
> > $6 = {cmd = 50, unicode = 1, request = 0, unidir = 0, info_level = -1,
> > info_count = -1, sip = 0xe7df048,
> >    ct = 0xe7a99a4}
> > (gdb) print *t2i
> > Cannot access memory at address 0x800a.
>
> That's probably a file ID.  The "extra_info" field of an
> "smb_saved_info_t" structure is usually a pointer to a
> SMB-message-type-specific data structure, but sometimes it's used to
> hold a numerical file ID instead.
>
> It looks as if a file ID was put there, but the code is assuming that
> it's not a file ID.
>
> _______________________________________________
> Ethereal-dev mailing list
> Ethereal-dev@xxxxxxxxxxxx
> http://www.ethereal.com/mailman/listinfo/ethereal-dev

Attachment: smb.patch
Description: Binary data