From 19c36f5ca3751f10d7065454c103c1d0356d8ada Mon Sep 17 00:00:00 2001 From: Daniel Stenberg Date: Thu, 13 Apr 2023 09:29:59 +0200 Subject: [PATCH] transfer: skip extra assign The 'result' variable already contains CURLE_OK at this point, no use in setting it again. Pointed out by PVS. Ref: #10929 Closes #10944 --- lib/transfer.c | 1 - 1 file changed, 1 deletion(-) diff --git a/lib/transfer.c b/lib/transfer.c index cb781833f7..e9ab8fbf09 100644 --- a/lib/transfer.c +++ b/lib/transfer.c @@ -1235,7 +1235,6 @@ CURLcode Curl_readwrite(struct connectdata *conn, /* Now update the "done" boolean we return */ *done = (0 == (k->keepon&(KEEP_RECVBITS|KEEP_SENDBITS))) ? TRUE : FALSE; - result = CURLE_OK; out: if(result) DEBUGF(infof(data, DMSG(data, "Curl_readwrite() -> %d"), result)); -- 2.47.3