CVS update: samba/source/rpc_parse
Gerald (Jerry) Carter
jerry at samba.org
Tue Jun 25 17:45:02 GMT 2002
On Wed, 26 Jun 2002, Tim Potter wrote:
> On Tue, Jun 25, 2002 at 05:15:16PM -0700, jerry at samba.org wrote:
>
> > Update of /data/cvs/samba/source/rpc_parse
> > In directory va:/tmp/cvs-serv15730/rpc_parse
> >
> > Modified Files:
> > Tag: APPLIANCE_HEAD
> > parse_misc.c parse_sec.c
> > Log Message:
> > removing all prs_align() calls from marshalling up security
> > descriptors. This appears to fix a failure in the PSA functionality
> > tests. Needs much more testing with file sec-desc to ensure
> > it hasn't broken anythign inadvertantly. That will come tomorrow. :-)
> >
> > Will merge into HEAD after I've spent more time in testing....
>
> 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.
cheers, jerry
More information about the samba-technical
mailing list