CVS update: samba/source/rpc_parse

Tim Potter tpot at samba.org
Tue Jun 25 18:04:01 GMT 2002


On Tue, Jun 25, 2002 at 07:39:12PM -0500, Gerald (Jerry) Carter wrote:

> > The change to parse_misc.c is OK - uint8's require no alignment.
> > 
> > However in parse_sec.c there should be alignments to a 2-byte boundary
> > before the prs_uint16 calls.  This probably just works anyway because
> > the offsets have probably been constructed with the correct alignment.
> > 
> > I think you still need the prs_align() for sec_io_access though.
> 
> Then the i goofed the size calculations.  I'll work on this some more 
> tomorrow.  Hopefully it won't get in our way until then.  Then problem
> was that the sec_desc was overwriting the first 2 bytes of the 
> of the device mode in the PrinterInfo(2) parsing....
> 
> Thanks for taking a look.  

Do you have a packet capture or a log file?  I'd be interested in taking
a look.


Tim.




More information about the samba-technical mailing list