Remove spurious checks in evrpc.c error cases (coverity)

Coverity hates the case where you have a redundant NULL check after
using something.
This commit is contained in:
Nick Mathewson 2014-01-08 12:04:06 -05:00
parent 544cf8888a
commit 991b362001

View File

@ -329,7 +329,6 @@ evrpc_request_cb(struct evhttp_request *req, void *arg)
return;
error:
if (rpc_state != NULL)
evrpc_reqstate_free_(rpc_state);
evhttp_send_error(req, HTTP_SERVUNAVAIL, NULL);
return;
@ -372,7 +371,6 @@ evrpc_request_cb_closure(void *arg, enum EVRPC_HOOK_RESULT hook_res)
return;
error:
if (rpc_state != NULL)
evrpc_reqstate_free_(rpc_state);
evhttp_send_error(req, HTTP_SERVUNAVAIL, NULL);
return;
@ -454,7 +452,6 @@ evrpc_request_done(struct evrpc_req_generic *rpc_state)
return;
error:
if (rpc_state != NULL)
evrpc_reqstate_free_(rpc_state);
evhttp_send_error(req, HTTP_SERVUNAVAIL, NULL);
return;