Remove NULL checks for bdrv_new return value

It's an indirect call to qemu_malloc, which never returns an error.

Signed-off-by: Kevin Wolf <kwolf@redhat.com>

diff --git a/hw/xen_disk.c b/hw/xen_disk.c
index 85a1c855246dbeff8cbe9f49eaafbdd9c5856674..ed9e5eb4d7ac56d6021390808ef16c3d932ffca4 100644 (file)
--- a/hw/xen_disk.c
+++ b/hw/xen_disk.c
@@ -634,17 +634,12 @@ static int blk_init(struct XenDevice *xendev)
     if (!blkdev->dinfo) {
         /* setup via xenbus -> create new block driver instance */
         xen_be_printf(&blkdev->xendev, 2, "create new bdrv (xenbus setup)\n");
-       blkdev->bs = bdrv_new(blkdev->dev);
-       if (blkdev->bs) {
-           if (bdrv_open(blkdev->bs, blkdev->filename, qflags,
-                           bdrv_find_whitelisted_format(blkdev->fileproto))
-                != 0) {
-               bdrv_delete(blkdev->bs);
-               blkdev->bs = NULL;
-           }
-       }
-       if (!blkdev->bs)
-           return -1;
+        blkdev->bs = bdrv_new(blkdev->dev);
+        if (bdrv_open(blkdev->bs, blkdev->filename, qflags,
+                      bdrv_find_whitelisted_format(blkdev->fileproto)) != 0) {
+            bdrv_delete(blkdev->bs);
+            return -1;
+        }
     } else {
         /* setup via qemu cmdline -> already setup for us */
         xen_be_printf(&blkdev->xendev, 2, "get configured bdrv (cmdline setup)\n");

diff --git a/qemu-img.c b/qemu-img.c
index 0b871d842c03b711984f771b3e15e19f338e06e8..afd9ed2e0ece60c22dbf025c4702365c9635687a 100644 (file)
--- a/qemu-img.c
+++ b/qemu-img.c
@@ -215,10 +215,7 @@ static BlockDriverState *bdrv_new_open(const char *filename,
     char password[256];
 
     bs = bdrv_new("");
-    if (!bs) {
-        error_report("Not enough memory");
-        goto fail;
-    }
+
     if (fmt) {
         drv = bdrv_find_format(fmt);
         if (!drv) {

diff --git a/qemu-io.c b/qemu-io.c
index ff353ebba358885764db7d77dfa7fc43fbf223a2..0f6d1b6b23dd1bc903dfc0e1fb6a9ce499877de0 100644 (file)
--- a/qemu-io.c
+++ b/qemu-io.c
@@ -1509,8 +1509,6 @@ static int openfile(char *name, int flags, int growable)
                }
        } else {
                bs = bdrv_new("hda");
-               if (!bs)
-                       return 1;
 
                if (bdrv_open(bs, name, flags, NULL) < 0) {
                        fprintf(stderr, "%s: can't open device %s\n", progname, name);

diff --git a/qemu-nbd.c b/qemu-nbd.c
index 99f1d22884c3aa339a2fcf4c08f942c903359d3c..e858033e06fdd5c44337eb7e90cd317955340d60 100644 (file)
--- a/qemu-nbd.c
+++ b/qemu-nbd.c
@@ -336,8 +336,6 @@ int main(int argc, char **argv)
     bdrv_init();
 
     bs = bdrv_new("hda");
-    if (bs == NULL)
-        return 1;
 
     if ((ret = bdrv_open(bs, argv[optind], flags, NULL)) < 0) {
         errno = -ret;