summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorVolker Lendecke <vl@samba.org>2011-03-21 10:56:10 +0100
committerVolker Lendecke <vlendec@samba.org>2011-03-21 16:29:22 +0100
commit16155812e04cee6542746b8dbb4a9a7acb93de70 (patch)
tree9c568569bcb4fa3cbb64e6788985ab7139a89960
parent8dc93bed09961ada9a25e8144cb761d1d7bd1ac5 (diff)
downloadsamba-16155812e04cee6542746b8dbb4a9a7acb93de70.tar.gz
samba-16155812e04cee6542746b8dbb4a9a7acb93de70.tar.bz2
samba-16155812e04cee6542746b8dbb4a9a7acb93de70.zip
s3: Fix Coverity ID 1048, CHECKED_RETURN
This is a real bug: tevent_req_set_endtime already calls tevent_req_nomem. Autobuild-User: Volker Lendecke <vlendec@samba.org> Autobuild-Date: Mon Mar 21 16:29:22 CET 2011 on sn-devel-104
-rw-r--r--source3/winbindd/winbindd_dual.c7
1 files changed, 1 insertions, 6 deletions
diff --git a/source3/winbindd/winbindd_dual.c b/source3/winbindd/winbindd_dual.c
index 47768731b7..2490073262 100644
--- a/source3/winbindd/winbindd_dual.c
+++ b/source3/winbindd/winbindd_dual.c
@@ -150,12 +150,7 @@ static void wb_child_request_trigger(struct tevent_req *req,
return;
}
tevent_req_set_callback(subreq, wb_child_request_done, req);
-
- if (!tevent_req_set_endtime(req, state->ev,
- timeval_current_ofs(300, 0))) {
- tevent_req_nomem(NULL, req);
- return;
- }
+ tevent_req_set_endtime(req, state->ev, timeval_current_ofs(300, 0));
}
static void wb_child_request_done(struct tevent_req *subreq)