summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRafal Szczesniak <mimir@samba.org>2006-12-12 22:34:35 +0000
committerGerald (Jerry) Carter <jerry@samba.org>2007-10-10 14:29:13 -0500
commit6de9ae9e43be1b2a765554a2c1d36226197e3cfa (patch)
tree5927646368676b65b0b78ec97984e9754ff8145c
parent3dde7ab2c0160aea5ccfe883176fd4869cb1b3a2 (diff)
downloadsamba-6de9ae9e43be1b2a765554a2c1d36226197e3cfa.tar.gz
samba-6de9ae9e43be1b2a765554a2c1d36226197e3cfa.tar.bz2
samba-6de9ae9e43be1b2a765554a2c1d36226197e3cfa.zip
r20138: return the proper status for the other functions as well.
rafal (This used to be commit 0b82b9dba1f67b1d340b7cf2b35a826135c101d4)
-rw-r--r--source4/libnet/userman.c10
1 files changed, 7 insertions, 3 deletions
diff --git a/source4/libnet/userman.c b/source4/libnet/userman.c
index c5bc8654c3..13834d2842 100644
--- a/source4/libnet/userman.c
+++ b/source4/libnet/userman.c
@@ -309,10 +309,13 @@ static NTSTATUS userdel_delete(struct composite_context *c,
/* receive samr_DeleteUser result */
c->status = dcerpc_ndr_request_recv(s->req);
NT_STATUS_NOT_OK_RETURN(c->status);
+
+ /* return the actual function call status */
+ c->status = s->deleteuser.out.result;
c->state = COMPOSITE_STATE_DONE;
- return NT_STATUS_OK;
+ return c->status;
}
@@ -778,7 +781,8 @@ static NTSTATUS usermod_modify(struct composite_context *c,
c->status = dcerpc_ndr_request_recv(s->req);
NT_STATUS_NOT_OK_RETURN(c->status);
- NT_STATUS_NOT_OK_RETURN(s->setuser.out.result);
+ /* return the actual function call status */
+ c->status = s->setuser.out.result;
if (s->change.fields == 0) {
/* all fields have been set - we're done */
@@ -788,7 +792,7 @@ static NTSTATUS usermod_modify(struct composite_context *c,
return usermod_change(c, s);
}
- return NT_STATUS_OK;
+ return c->status;
}