diff options
| author | Paolo Bonzini <pbonzini@redhat.com> | 2015-09-14 13:12:34 +0200 |
|---|---|---|
| committer | Michael Tokarev <mjt@tls.msk.ru> | 2015-10-08 19:46:01 +0300 |
| commit | eab2ac9d3c1675a58989000c2647aa33e440906a (patch) | |
| tree | 96b48138e7d10678b42dcb0faca8b25405f12fd4 | |
| parent | dc1442204a2235b1ad0c4bdceb3580c97f71f1b5 (diff) | |
block/ssh: remove dead code
The "err" label cannot be reached with qp != NULL. Remove the free-ing
of qp and avoid future regressions by removing the initializer.
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
ACKed-by: Richard W.M. Jones <rjones@redhat.com>
Reviewed-by: Fam Zheng <famz@redhat.com>
Signed-off-by: Michael Tokarev <mjt@tls.msk.ru>
| -rw-r--r-- | block/ssh.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/block/ssh.c b/block/ssh.c index 8d0673903d..d35b51f987 100644 --- a/block/ssh.c +++ b/block/ssh.c @@ -193,7 +193,7 @@ sftp_error_report(BDRVSSHState *s, const char *fs, ...) static int parse_uri(const char *filename, QDict *options, Error **errp) { URI *uri = NULL; - QueryParams *qp = NULL; + QueryParams *qp; int i; uri = uri_parse(filename); @@ -249,9 +249,6 @@ static int parse_uri(const char *filename, QDict *options, Error **errp) return 0; err: - if (qp) { - query_params_free(qp); - } if (uri) { uri_free(uri); } |