diff options
| author | Markus Armbruster | 2017-03-06 20:00:37 +0100 |
|---|---|---|
| committer | Kevin Wolf | 2017-03-07 14:53:28 +0100 |
| commit | 48d7c4af06393b974b0a515ac9d1cc7346dbae23 (patch) | |
| tree | ac00b637c94c204f4d9eed38efac920363fca552 /scripts/switch-timer-api | |
| parent | sheepdog: Fix error handling in sd_snapshot_delete() (diff) | |
| download | qemu-48d7c4af06393b974b0a515ac9d1cc7346dbae23.tar.gz qemu-48d7c4af06393b974b0a515ac9d1cc7346dbae23.tar.xz qemu-48d7c4af06393b974b0a515ac9d1cc7346dbae23.zip | |
sheepdog: Fix error handling sd_create()
As a bdrv_create() method, sd_create() must set an error and return
negative errno on failure. It prints the error instead of setting it
when connect_to_sdog() fails. Fix that.
While there, return the value of connect_to_sdog() like we do
elsewhere, instead of -EIO. No functional change, as
connect_to_sdog() returns no other error code.
Many more suspicious uses of error_report() and error_report_err()
remain in other functions. Left for another day.
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Reviewed-by: Kevin Wolf <kwolf@redhat.com>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
Diffstat (limited to 'scripts/switch-timer-api')
0 files changed, 0 insertions, 0 deletions
