r5646: state->loadfile might be NULL after allocation so this is really
authorAlexander Bokovoy <ab@samba.org>
Thu, 3 Mar 2005 09:26:08 +0000 (09:26 +0000)
committerGerald (Jerry) Carter <jerry@samba.org>
Wed, 10 Oct 2007 18:10:56 +0000 (13:10 -0500)
should be NT_STATUS_HAVE_NO_MEMORY(state->loadfile) instead of
NT_STATUS_NOT_OK_RETURN(status).
(This used to be commit eb57a587889611bcf39d75d4e15b627f36899a53)

source4/libcli/composite/fetchfile.c

index a7d6f2ffd87bff42f247a2be8c9f3fd561315deb..2bf6ef902321bc4a241d29d1187230df5f29bab0 100644 (file)
@@ -49,7 +49,7 @@ static NTSTATUS fetchfile_connect(struct composite_context *c,
        NT_STATUS_NOT_OK_RETURN(status);
 
        state->loadfile = talloc(state, struct smb_composite_loadfile);
-       NT_STATUS_NOT_OK_RETURN(status);
+       NT_STATUS_HAVE_NO_MEMORY(state->loadfile);
 
        state->loadfile->in.fname = io->in.filename;