Padding byte in cifs readx response
jra at samba.org
Fri Aug 15 11:24:22 MDT 2014
On Fri, Aug 15, 2014 at 07:22:16PM +0200, Volker Lendecke wrote:
> On Fri, Aug 15, 2014 at 10:17:15AM -0700, Jeremy Allison wrote:
> > On Thu, Aug 14, 2014 at 10:30:52PM -0700, Christof Schmitt wrote:
> > > On Wed, Aug 13, 2014 at 04:10:34PM +0200, Volker Lendecke wrote:
> > > > We just overflowed the 16MB nbss packet. I've attached one
> > > > question, a few R-Bs and a possible fix.
> > >
> > > Thanks. The fix looks good, i included it in the patch series, see
> > > attachment.
> > > >
> > > > VL: I think at least in the aio case the padding byte is left uninitialized.
> > > > Can you check that? Thanks!
> > >
> > > Yes, i missed that. struct aio_extra is allocated and zeroed, but not
> > > the following data buffer. I added the explicit initialization of the
> > > padding byte.
> > LGTM Christof thanks ! Pushed.
> Wait please!
> I'm still dubious about the new_size += 1 in
> smb_splice_chain. Can you explain that? I'm really nervous
> about that piece of the code, &x is really from hell.
Ok, will hold off on that :-).
andX is indeed from the darkest pits of hades. But
I thought you'd looked at that already - my mistake,
More information about the samba-technical