[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH 6/7] block: Use error codes from lower levels for er
From: |
Kevin Wolf |
Subject: |
[Qemu-devel] [PATCH 6/7] block: Use error codes from lower levels for error message |
Date: |
Mon, 26 Jul 2010 16:01:16 +0200 |
From: Stefan Weil <address@hidden>
"No such file or directory" is a misleading error message
when a user tries to open a file with wrong permissions.
Cc: Kevin Wolf <address@hidden>
Signed-off-by: Stefan Weil <address@hidden>
Signed-off-by: Kevin Wolf <address@hidden>
---
block.c | 27 +++++++++++++++++++--------
1 files changed, 19 insertions(+), 8 deletions(-)
diff --git a/block.c b/block.c
index f837876..49e6cbc 100644
--- a/block.c
+++ b/block.c
@@ -330,7 +330,7 @@ BlockDriver *bdrv_find_protocol(const char *filename)
return NULL;
}
-static BlockDriver *find_image_format(const char *filename)
+static int find_image_format(const char *filename, BlockDriver **pdrv)
{
int ret, score, score_max;
BlockDriver *drv1, *drv;
@@ -338,19 +338,27 @@ static BlockDriver *find_image_format(const char
*filename)
BlockDriverState *bs;
ret = bdrv_file_open(&bs, filename, 0);
- if (ret < 0)
- return NULL;
+ if (ret < 0) {
+ *pdrv = NULL;
+ return ret;
+ }
/* Return the raw BlockDriver * to scsi-generic devices or empty drives */
if (bs->sg || !bdrv_is_inserted(bs)) {
bdrv_delete(bs);
- return bdrv_find_format("raw");
+ drv = bdrv_find_format("raw");
+ if (!drv) {
+ ret = -ENOENT;
+ }
+ *pdrv = drv;
+ return ret;
}
ret = bdrv_pread(bs, 0, buf, sizeof(buf));
bdrv_delete(bs);
if (ret < 0) {
- return NULL;
+ *pdrv = NULL;
+ return ret;
}
score_max = 0;
@@ -364,7 +372,11 @@ static BlockDriver *find_image_format(const char *filename)
}
}
}
- return drv;
+ if (!drv) {
+ ret = -ENOENT;
+ }
+ *pdrv = drv;
+ return ret;
}
/**
@@ -571,12 +583,11 @@ int bdrv_open(BlockDriverState *bs, const char *filename,
int flags,
/* Find the right image format driver */
if (!drv) {
- drv = find_image_format(filename);
+ ret = find_image_format(filename, &drv);
probed = 1;
}
if (!drv) {
- ret = -ENOENT;
goto unlink_and_fail;
}
--
1.7.1.1
- [Qemu-devel] [PULL 0/7] Block patches, Kevin Wolf, 2010/07/26
- [Qemu-devel] [PATCH 1/7] ide/atapi: add support for GET EVENT STATUS NOTIFICATION, Kevin Wolf, 2010/07/26
- [Qemu-devel] [PATCH 2/7] block migration: propagate return value when bdrv_write() returns < 0, Kevin Wolf, 2010/07/26
- [Qemu-devel] [PATCH 3/7] virtio-blk: Create exit function to unregister savevm, Kevin Wolf, 2010/07/26
- [Qemu-devel] [PATCH 7/7] Fix -snapshot deleting images on disk change, Kevin Wolf, 2010/07/26
- [Qemu-devel] [PATCH 5/7] block: default to 0 minimal / optiomal I/O size, Kevin Wolf, 2010/07/26
- [Qemu-devel] [PATCH 4/7] move 'unsafe' to end of caching modes in help, Kevin Wolf, 2010/07/26
- [Qemu-devel] [PATCH 6/7] block: Use error codes from lower levels for error message,
Kevin Wolf <=