diff options
author | Max Reitz <mreitz@redhat.com> | 2017-03-31 14:04:31 +0200 |
---|---|---|
committer | Jeff Cody <jcody@redhat.com> | 2017-03-31 15:53:22 -0400 |
commit | 34634ca28688652198f77d7001c0f1e204434663 (patch) | |
tree | 83fae1f529bef142b654076f6862ef1419c31f53 /block | |
parent | 6b9d62db89a31882d53d8fbfb67d4d82f22877b6 (diff) | |
download | qemu-34634ca28688652198f77d7001c0f1e204434663.zip qemu-34634ca28688652198f77d7001c0f1e204434663.tar.gz qemu-34634ca28688652198f77d7001c0f1e204434663.tar.bz2 |
block/curl: Check protocol prefix
If the user has explicitly specified a block driver and thus a protocol,
we have to make sure the URL's protocol prefix matches. Otherwise the
latter will silently override the former which might catch some users by
surprise.
Signed-off-by: Max Reitz <mreitz@redhat.com>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Reviewed-by: Jeff Cody <jcody@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Message-id: 20170331120431.1767-3-mreitz@redhat.com
Signed-off-by: Jeff Cody <jcody@redhat.com>
Diffstat (limited to 'block')
-rw-r--r-- | block/curl.c | 10 |
1 files changed, 10 insertions, 0 deletions
diff --git a/block/curl.c b/block/curl.c index 34dbd33..2708d57 100644 --- a/block/curl.c +++ b/block/curl.c @@ -659,6 +659,7 @@ static int curl_open(BlockDriverState *bs, QDict *options, int flags, const char *cookie; double d; const char *secretid; + const char *protocol_delimiter; static int inited = 0; @@ -700,6 +701,15 @@ static int curl_open(BlockDriverState *bs, QDict *options, int flags, goto out_noclean; } + if (!strstart(file, bs->drv->protocol_name, &protocol_delimiter) || + !strstart(protocol_delimiter, "://", NULL)) + { + error_setg(errp, "%s curl driver cannot handle the URL '%s' (does not " + "start with '%s://')", bs->drv->protocol_name, file, + bs->drv->protocol_name); + goto out_noclean; + } + s->username = g_strdup(qemu_opt_get(opts, CURL_BLOCK_OPT_USERNAME)); secretid = qemu_opt_get(opts, CURL_BLOCK_OPT_PASSWORD_SECRET); |