From 522e647cc52c45ebdb58d57f242204f9a72c45dd Mon Sep 17 00:00:00 2001
From: Daniel Gustafsson <daniel@yesql.se>
Date: Wed, 19 Sep 2018 13:44:10 +0200
Subject: urlapi: don't set value which is never read

In the CURLUPART_URL case, there is no codepath which invokes url
decoding so remove the assignment of the urldecode variable. This
fixes the deadstore bug-report from clang static analysis.

Closes #3015
Reviewed-by: Daniel Stenberg <daniel@haxx.se>
---
 lib/urlapi.c | 1 -
 1 file changed, 1 deletion(-)

(limited to 'lib')

diff --git a/lib/urlapi.c b/lib/urlapi.c
index 0ac550741..f6d911667 100644
--- a/lib/urlapi.c
+++ b/lib/urlapi.c
@@ -970,7 +970,6 @@ CURLUcode curl_url_get(CURLU *u, CURLUPart what,
     char *scheme;
     char *options = u->options;
     char *port = u->port;
-    urldecode = FALSE; /* not for the whole thing */
     if(u->scheme && strcasecompare("file", u->scheme)) {
       url = aprintf("file://%s%s%s",
                     u->path,
-- 
cgit v1.2.3