diff options
| author | Jeremy Allison <jra@samba.org> | 2011-07-29 15:20:15 -0700 |
|---|---|---|
| committer | Jeremy Allison <jra@samba.org> | 2011-07-30 01:34:24 +0200 |
| commit | 20493572178ed5791806dbdb4572f595db3110b2 (patch) | |
| tree | db6372a4b8bd0f62d2f568f66289fbf6315fc33e | |
| parent | 1d4d9194abf3694e2f890744c6f7764088b1c1f3 (diff) | |
| download | samba-20493572178ed5791806dbdb4572f595db3110b2.tar.gz samba-20493572178ed5791806dbdb4572f595db3110b2.tar.bz2 samba-20493572178ed5791806dbdb4572f595db3110b2.zip | |
Added missing return. Not strictly neccessary, the following if (!NT_STATUS_EQUAL(status, NT_STATUS_NOT_SUPPORTED))
clause will catch this and use tevent_req_nterror(req, status); to set the status as NT_STATUS_OK, but
it looks strange to do it that way.
Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Sat Jul 30 01:34:24 CEST 2011 on sn-devel-104
| -rw-r--r-- | source3/libsmb/clifile.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/source3/libsmb/clifile.c b/source3/libsmb/clifile.c index 3590350559..59cdb7b855 100644 --- a/source3/libsmb/clifile.c +++ b/source3/libsmb/clifile.c @@ -2253,6 +2253,7 @@ static void cli_open_done(struct tevent_req *subreq) if (NT_STATUS_IS_OK(status)) { state->fnum = SVAL(vwv+2, 0); tevent_req_done(req); + return; } if (!NT_STATUS_EQUAL(status, NT_STATUS_NOT_SUPPORTED)) { |