diff options
| author | Daniel Gustafsson <daniel@yesql.se> | 2018-09-19 13:44:10 +0200 |
|---|---|---|
| committer | Daniel Gustafsson <daniel@yesql.se> | 2018-09-19 13:44:10 +0200 |
| commit | 522e647cc52c45ebdb58d57f242204f9a72c45dd (patch) | |
| tree | b4cba63aee9667de119f5f1082ede16dc268aa38 /lib/urlapi.c | |
| parent | 056cc37ed4eb1c5905e3732716e3a3d44039e6a8 (diff) | |
| download | curl-522e647cc52c45ebdb58d57f242204f9a72c45dd.tar.gz | |
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>
Diffstat (limited to 'lib/urlapi.c')
| -rw-r--r-- | lib/urlapi.c | 1 |
1 files changed, 0 insertions, 1 deletions
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, |