[PATCH] smb2_create: move 'op' into scope (Re: [PATCHES] smb3 create replay (for multi-channel))
obnox at samba.org
Thu Mar 3 08:43:24 UTC 2016
On 2016-03-03 at 09:33 +0100, Michael Adam wrote:
> On 2016-03-02 at 16:45 -0800, Jeremy Allison wrote:
> > On Wed, Mar 02, 2016 at 10:45:11PM +0100, Michael Adam wrote:
> > >
> > > Attached find the updated patchset.
> > > Differences with the original one:
> > Pushing this, but autobuild is failing with:
> > [252(789)/1892 at 20m43s] samba3.blackbox.smbclient_machine_auth.plain (nt4_member:local)
> > ERROR: Testsuite[samba3.blackbox.smbclient_machine_auth.plain (nt4_member:local)]
> > REASON: unable to set up environment nt4_member:local - exiting
> > Don't think this is to do with your new code.
> There was a NULL dereference of 'op' in smb2_create_send
> right at the and, because I used op instead of result->op
> in one (new) place. Usually these are the same at this
> place, but for IPC or PRINT shares, we ended up here without
> op == result->op but op == NULL.
> Attached find the 2-line patch to be squashed into the
> 'implement create replay' test to fix the crash.
> I am running a full make test with this now and will
> (given the in-retrospect obviousness) push once this has
Attached find a patch that would have prevented such
pitfall by moving 'op' into the scope of its use.
Note: due to context change, it does not apply to current
master but only on top of the create-replay patchset.
Cheers - Michael
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 198 bytes
Desc: not available
More information about the samba-technical