From 20493572178ed5791806dbdb4572f595db3110b2 Mon Sep 17 00:00:00 2001 From: Jeremy Allison Date: Fri, 29 Jul 2011 15:20:15 -0700 Subject: 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 Autobuild-Date: Sat Jul 30 01:34:24 CEST 2011 on sn-devel-104 --- source3/libsmb/clifile.c | 1 + 1 file changed, 1 insertion(+) 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)) { -- cgit