From 7a8cc11406db7f3ea70425b71e31ece0b07e9e08 Mon Sep 17 00:00:00 2001 From: Azat Khuzhin Date: Mon, 1 Apr 2019 02:37:41 +0300 Subject: [PATCH] evrpc: do not check req twice MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit reported by µchex: "In evrpc.c, pointer req is dereferenced on line 881 before it is null checked on line 894." --- evrpc.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/evrpc.c b/evrpc.c index 3b5260fc..46bf3f7e 100644 --- a/evrpc.c +++ b/evrpc.c @@ -892,8 +892,7 @@ evrpc_reply_done(struct evhttp_request *req, void *arg) * layer is going to free it. we need to * request ownership explicitly */ - if (req != NULL) - evhttp_request_own(req); + evhttp_request_own(req); evrpc_pause_request(pool, ctx, evrpc_reply_done_closure); -- 2.50.1