diff options
| author | Peter Maydell | 2022-02-22 16:23:41 +0100 |
|---|---|---|
| committer | Hanna Reitz | 2022-03-07 09:19:20 +0100 |
| commit | b0ea6c98fa2431b9e24b3a53b8b42c960ccf3dba (patch) | |
| tree | 6dfeeafc8a3b8dcf14601b8e207205f7cf290588 /include/block | |
| parent | block/curl.c: Set error message string if curl_init_state() fails (diff) | |
| download | qemu-b0ea6c98fa2431b9e24b3a53b8b42c960ccf3dba.tar.gz qemu-b0ea6c98fa2431b9e24b3a53b8b42c960ccf3dba.tar.xz qemu-b0ea6c98fa2431b9e24b3a53b8b42c960ccf3dba.zip | |
block/curl.c: Check error return from curl_easy_setopt()
Coverity points out that we aren't checking the return value
from curl_easy_setopt() for any of the calls to it we make
in block/curl.c.
Some of these options are documented as always succeeding (e.g.
CURLOPT_VERBOSE) but others have documented failure cases (e.g.
CURLOPT_URL). For consistency we check every call, even the ones
that theoretically cannot fail.
Fixes: Coverity CID 1459336, 1459482, 1460331
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Message-Id: <20220222152341.850419-3-peter.maydell@linaro.org>
Reviewed-by: Hanna Reitz <hreitz@redhat.com>
Signed-off-by: Hanna Reitz <hreitz@redhat.com>
Diffstat (limited to 'include/block')
0 files changed, 0 insertions, 0 deletions