cryptospore / rpms / qemu-kvm

Forked from rpms/qemu-kvm 2 years ago
Clone
218e99
From 9f0ec0582867ea7c45ac5b23bfcbd20e699a4bd4 Mon Sep 17 00:00:00 2001
218e99
From: Max Reitz <mreitz@redhat.com>
218e99
Date: Mon, 4 Nov 2013 22:32:14 +0100
218e99
Subject: [PATCH 21/87] qcow2: Use Error parameter
218e99
218e99
RH-Author: Max Reitz <mreitz@redhat.com>
218e99
Message-id: <1383604354-12743-24-git-send-email-mreitz@redhat.com>
218e99
Patchwork-id: 55323
218e99
O-Subject: [RHEL-7.0 qemu-kvm PATCH 23/43] qcow2: Use Error parameter
218e99
Bugzilla: 1026524
218e99
RH-Acked-by: Kevin Wolf <kwolf@redhat.com>
218e99
RH-Acked-by: Laszlo Ersek <lersek@redhat.com>
218e99
RH-Acked-by: Fam Zheng <famz@redhat.com>
218e99
RH-Acked-by: Stefan Hajnoczi <stefanha@redhat.com>
218e99
218e99
BZ: 1026524
218e99
218e99
Employ usage of the new Error ** parameter in qcow2_open, qcow2_create
218e99
and associated functions.
218e99
218e99
Signed-off-by: Max Reitz <mreitz@redhat.com>
218e99
(cherry picked from commit 3ef6c40ad0b350e18c78135ffbdbe209cb479c1f)
218e99
218e99
Signed-off-by: Max Reitz <mreitz@redhat.com>
218e99
218e99
Conflicts:
218e99
	block/qcow2.c
218e99
218e99
Conflicts because 8ad1898c has not been backported ("qcow2: Change
218e99
default for new images to compat=1.1").
218e99
---
218e99
 block/qcow2.c | 134 ++++++++++++++++++++++++++++++++++++++--------------------
218e99
 1 file changed, 88 insertions(+), 46 deletions(-)
218e99
218e99
Signed-off-by: Miroslav Rezanina <mrezanin@redhat.com>
218e99
---
218e99
 block/qcow2.c |  134 +++++++++++++++++++++++++++++++++++++-------------------
218e99
 1 files changed, 88 insertions(+), 46 deletions(-)
218e99
218e99
diff --git a/block/qcow2.c b/block/qcow2.c
218e99
index 027d210..880d2cf 100644
218e99
--- a/block/qcow2.c
218e99
+++ b/block/qcow2.c
218e99
@@ -79,7 +79,8 @@ static int qcow2_probe(const uint8_t *buf, int buf_size, const char *filename)
218e99
  * return 0 upon success, non-0 otherwise
218e99
  */
218e99
 static int qcow2_read_extensions(BlockDriverState *bs, uint64_t start_offset,
218e99
-                                 uint64_t end_offset, void **p_feature_table)
218e99
+                                 uint64_t end_offset, void **p_feature_table,
218e99
+                                 Error **errp)
218e99
 {
218e99
     BDRVQcowState *s = bs->opaque;
218e99
     QCowExtension ext;
218e99
@@ -100,10 +101,10 @@ static int qcow2_read_extensions(BlockDriverState *bs, uint64_t start_offset,
218e99
         printf("attempting to read extended header in offset %lu\n", offset);
218e99
 #endif
218e99
 
218e99
-        if (bdrv_pread(bs->file, offset, &ext, sizeof(ext)) != sizeof(ext)) {
218e99
-            fprintf(stderr, "qcow2_read_extension: ERROR: "
218e99
-                    "pread fail from offset %" PRIu64 "\n",
218e99
-                    offset);
218e99
+        ret = bdrv_pread(bs->file, offset, &ext, sizeof(ext));
218e99
+        if (ret < 0) {
218e99
+            error_setg_errno(errp, -ret, "qcow2_read_extension: ERROR: "
218e99
+                             "pread fail from offset %" PRIu64, offset);
218e99
             return 1;
218e99
         }
218e99
         be32_to_cpus(&ext.magic);
218e99
@@ -113,7 +114,7 @@ static int qcow2_read_extensions(BlockDriverState *bs, uint64_t start_offset,
218e99
         printf("ext.magic = 0x%x\n", ext.magic);
218e99
 #endif
218e99
         if (ext.len > end_offset - offset) {
218e99
-            error_report("Header extension too large");
218e99
+            error_setg(errp, "Header extension too large");
218e99
             return -EINVAL;
218e99
         }
218e99
 
218e99
@@ -123,14 +124,16 @@ static int qcow2_read_extensions(BlockDriverState *bs, uint64_t start_offset,
218e99
 
218e99
         case QCOW2_EXT_MAGIC_BACKING_FORMAT:
218e99
             if (ext.len >= sizeof(bs->backing_format)) {
218e99
-                fprintf(stderr, "ERROR: ext_backing_format: len=%u too large"
218e99
-                        " (>=%zu)\n",
218e99
-                        ext.len, sizeof(bs->backing_format));
218e99
+                error_setg(errp, "ERROR: ext_backing_format: len=%u too large"
218e99
+                           " (>=%zu)", ext.len, sizeof(bs->backing_format));
218e99
                 return 2;
218e99
             }
218e99
-            if (bdrv_pread(bs->file, offset , bs->backing_format,
218e99
-                           ext.len) != ext.len)
218e99
+            ret = bdrv_pread(bs->file, offset, bs->backing_format, ext.len);
218e99
+            if (ret < 0) {
218e99
+                error_setg_errno(errp, -ret, "ERROR: ext_backing_format: "
218e99
+                                 "Could not read format name");
218e99
                 return 3;
218e99
+            }
218e99
             bs->backing_format[ext.len] = '\0';
218e99
 #ifdef DEBUG_EXT
218e99
             printf("Qcow2: Got format extension %s\n", bs->backing_format);
218e99
@@ -142,6 +145,8 @@ static int qcow2_read_extensions(BlockDriverState *bs, uint64_t start_offset,
218e99
                 void* feature_table = g_malloc0(ext.len + 2 * sizeof(Qcow2Feature));
218e99
                 ret = bdrv_pread(bs->file, offset , feature_table, ext.len);
218e99
                 if (ret < 0) {
218e99
+                    error_setg_errno(errp, -ret, "ERROR: ext_feature_table: "
218e99
+                                     "Could not read table");
218e99
                     return ret;
218e99
                 }
218e99
 
218e99
@@ -161,6 +166,8 @@ static int qcow2_read_extensions(BlockDriverState *bs, uint64_t start_offset,
218e99
 
218e99
                 ret = bdrv_pread(bs->file, offset , uext->data, uext->len);
218e99
                 if (ret < 0) {
218e99
+                    error_setg_errno(errp, -ret, "ERROR: unknown extension: "
218e99
+                                     "Could not read data");
218e99
                     return ret;
218e99
                 }
218e99
             }
218e99
@@ -184,8 +191,8 @@ static void cleanup_unknown_header_ext(BlockDriverState *bs)
218e99
     }
218e99
 }
218e99
 
218e99
-static void GCC_FMT_ATTR(2, 3) report_unsupported(BlockDriverState *bs,
218e99
-    const char *fmt, ...)
218e99
+static void GCC_FMT_ATTR(3, 4) report_unsupported(BlockDriverState *bs,
218e99
+    Error **errp, const char *fmt, ...)
218e99
 {
218e99
     char msg[64];
218e99
     va_list ap;
218e99
@@ -194,17 +201,17 @@ static void GCC_FMT_ATTR(2, 3) report_unsupported(BlockDriverState *bs,
218e99
     vsnprintf(msg, sizeof(msg), fmt, ap);
218e99
     va_end(ap);
218e99
 
218e99
-    qerror_report(QERR_UNKNOWN_BLOCK_FORMAT_FEATURE,
218e99
-        bs->device_name, "qcow2", msg);
218e99
+    error_set(errp, QERR_UNKNOWN_BLOCK_FORMAT_FEATURE, bs->device_name, "qcow2",
218e99
+              msg);
218e99
 }
218e99
 
218e99
 static void report_unsupported_feature(BlockDriverState *bs,
218e99
-    Qcow2Feature *table, uint64_t mask)
218e99
+    Error **errp, Qcow2Feature *table, uint64_t mask)
218e99
 {
218e99
     while (table && table->name[0] != '\0') {
218e99
         if (table->type == QCOW2_FEAT_TYPE_INCOMPATIBLE) {
218e99
             if (mask & (1 << table->bit)) {
218e99
-                report_unsupported(bs, "%.46s",table->name);
218e99
+                report_unsupported(bs, errp, "%.46s", table->name);
218e99
                 mask &= ~(1 << table->bit);
218e99
             }
218e99
         }
218e99
@@ -212,7 +219,8 @@ static void report_unsupported_feature(BlockDriverState *bs,
218e99
     }
218e99
 
218e99
     if (mask) {
218e99
-        report_unsupported(bs, "Unknown incompatible feature: %" PRIx64, mask);
218e99
+        report_unsupported(bs, errp, "Unknown incompatible feature: %" PRIx64,
218e99
+                           mask);
218e99
     }
218e99
 }
218e99
 
218e99
@@ -363,6 +371,7 @@ static int qcow2_open(BlockDriverState *bs, QDict *options, int flags,
218e99
 
218e99
     ret = bdrv_pread(bs->file, 0, &header, sizeof(header));
218e99
     if (ret < 0) {
218e99
+        error_setg_errno(errp, -ret, "Could not read qcow2 header");
218e99
         goto fail;
218e99
     }
218e99
     be32_to_cpus(&header.magic);
218e99
@@ -380,11 +389,12 @@ static int qcow2_open(BlockDriverState *bs, QDict *options, int flags,
218e99
     be32_to_cpus(&header.nb_snapshots);
218e99
 
218e99
     if (header.magic != QCOW_MAGIC) {
218e99
+        error_setg(errp, "Image is not in qcow2 format");
218e99
         ret = -EMEDIUMTYPE;
218e99
         goto fail;
218e99
     }
218e99
     if (header.version < 2 || header.version > 3) {
218e99
-        report_unsupported(bs, "QCOW version %d", header.version);
218e99
+        report_unsupported(bs, errp, "QCOW version %d", header.version);
218e99
         ret = -ENOTSUP;
218e99
         goto fail;
218e99
     }
218e99
@@ -412,6 +422,8 @@ static int qcow2_open(BlockDriverState *bs, QDict *options, int flags,
218e99
         ret = bdrv_pread(bs->file, sizeof(header), s->unknown_header_fields,
218e99
                          s->unknown_header_fields_size);
218e99
         if (ret < 0) {
218e99
+            error_setg_errno(errp, -ret, "Could not read unknown qcow2 header "
218e99
+                             "fields");
218e99
             goto fail;
218e99
         }
218e99
     }
218e99
@@ -430,8 +442,8 @@ static int qcow2_open(BlockDriverState *bs, QDict *options, int flags,
218e99
     if (s->incompatible_features & ~QCOW2_INCOMPAT_MASK) {
218e99
         void *feature_table = NULL;
218e99
         qcow2_read_extensions(bs, header.header_length, ext_end,
218e99
-                              &feature_table);
218e99
-        report_unsupported_feature(bs, feature_table,
218e99
+                              &feature_table, NULL);
218e99
+        report_unsupported_feature(bs, errp, feature_table,
218e99
                                    s->incompatible_features &
218e99
                                    ~QCOW2_INCOMPAT_MASK);
218e99
         ret = -ENOTSUP;
218e99
@@ -442,8 +454,8 @@ static int qcow2_open(BlockDriverState *bs, QDict *options, int flags,
218e99
         /* Corrupt images may not be written to unless they are being repaired
218e99
          */
218e99
         if ((flags & BDRV_O_RDWR) && !(flags & BDRV_O_CHECK)) {
218e99
-            error_report("qcow2: Image is corrupt; cannot be opened "
218e99
-                    "read/write.");
218e99
+            error_setg(errp, "qcow2: Image is corrupt; cannot be opened "
218e99
+                       "read/write");
218e99
             ret = -EACCES;
218e99
             goto fail;
218e99
         }
218e99
@@ -451,7 +463,7 @@ static int qcow2_open(BlockDriverState *bs, QDict *options, int flags,
218e99
 
218e99
     /* Check support for various header values */
218e99
     if (header.refcount_order != 4) {
218e99
-        report_unsupported(bs, "%d bit reference counts",
218e99
+        report_unsupported(bs, errp, "%d bit reference counts",
218e99
                            1 << header.refcount_order);
218e99
         ret = -ENOTSUP;
218e99
         goto fail;
218e99
@@ -459,10 +471,13 @@ static int qcow2_open(BlockDriverState *bs, QDict *options, int flags,
218e99
 
218e99
     if (header.cluster_bits < MIN_CLUSTER_BITS ||
218e99
         header.cluster_bits > MAX_CLUSTER_BITS) {
218e99
+        error_setg(errp, "Unsupported cluster size: 2^%i", header.cluster_bits);
218e99
         ret = -EINVAL;
218e99
         goto fail;
218e99
     }
218e99
     if (header.crypt_method > QCOW_CRYPT_AES) {
218e99
+        error_setg(errp, "Unsupported encryption method: %i",
218e99
+                   header.crypt_method);
218e99
         ret = -EINVAL;
218e99
         goto fail;
218e99
     }
218e99
@@ -491,6 +506,7 @@ static int qcow2_open(BlockDriverState *bs, QDict *options, int flags,
218e99
 
218e99
     l1_vm_state_index = size_to_l1(s, header.size);
218e99
     if (l1_vm_state_index > INT_MAX) {
218e99
+        error_setg(errp, "Image is too big");
218e99
         ret = -EFBIG;
218e99
         goto fail;
218e99
     }
218e99
@@ -499,6 +515,7 @@ static int qcow2_open(BlockDriverState *bs, QDict *options, int flags,
218e99
     /* the L1 table must contain at least enough entries to put
218e99
        header.size bytes */
218e99
     if (s->l1_size < s->l1_vm_state_index) {
218e99
+        error_setg(errp, "L1 table is too small");
218e99
         ret = -EINVAL;
218e99
         goto fail;
218e99
     }
218e99
@@ -509,6 +526,7 @@ static int qcow2_open(BlockDriverState *bs, QDict *options, int flags,
218e99
         ret = bdrv_pread(bs->file, s->l1_table_offset, s->l1_table,
218e99
                          s->l1_size * sizeof(uint64_t));
218e99
         if (ret < 0) {
218e99
+            error_setg_errno(errp, -ret, "Could not read L1 table");
218e99
             goto fail;
218e99
         }
218e99
         for(i = 0;i < s->l1_size; i++) {
218e99
@@ -529,6 +547,7 @@ static int qcow2_open(BlockDriverState *bs, QDict *options, int flags,
218e99
 
218e99
     ret = qcow2_refcount_init(bs);
218e99
     if (ret != 0) {
218e99
+        error_setg_errno(errp, -ret, "Could not initialize refcount handling");
218e99
         goto fail;
218e99
     }
218e99
 
218e99
@@ -536,7 +555,9 @@ static int qcow2_open(BlockDriverState *bs, QDict *options, int flags,
218e99
     QTAILQ_INIT(&s->discards);
218e99
 
218e99
     /* read qcow2 extensions */
218e99
-    if (qcow2_read_extensions(bs, header.header_length, ext_end, NULL)) {
218e99
+    if (qcow2_read_extensions(bs, header.header_length, ext_end, NULL,
218e99
+        &local_err)) {
218e99
+        error_propagate(errp, local_err);
218e99
         ret = -EINVAL;
218e99
         goto fail;
218e99
     }
218e99
@@ -550,6 +571,7 @@ static int qcow2_open(BlockDriverState *bs, QDict *options, int flags,
218e99
         ret = bdrv_pread(bs->file, header.backing_file_offset,
218e99
                          bs->backing_file, len);
218e99
         if (ret < 0) {
218e99
+            error_setg_errno(errp, -ret, "Could not read backing file name");
218e99
             goto fail;
218e99
         }
218e99
         bs->backing_file[len] = '\0';
218e99
@@ -557,6 +579,7 @@ static int qcow2_open(BlockDriverState *bs, QDict *options, int flags,
218e99
 
218e99
     ret = qcow2_read_snapshots(bs);
218e99
     if (ret < 0) {
218e99
+        error_setg_errno(errp, -ret, "Could not read snapshots");
218e99
         goto fail;
218e99
     }
218e99
 
218e99
@@ -565,6 +588,7 @@ static int qcow2_open(BlockDriverState *bs, QDict *options, int flags,
218e99
         s->autoclear_features = 0;
218e99
         ret = qcow2_update_header(bs);
218e99
         if (ret < 0) {
218e99
+            error_setg_errno(errp, -ret, "Could not update qcow2 header");
218e99
             goto fail;
218e99
         }
218e99
     }
218e99
@@ -579,6 +603,7 @@ static int qcow2_open(BlockDriverState *bs, QDict *options, int flags,
218e99
 
218e99
         ret = qcow2_check(bs, &result, BDRV_FIX_ERRORS);
218e99
         if (ret < 0) {
218e99
+            error_setg_errno(errp, -ret, "Could not repair dirty image");
218e99
             goto fail;
218e99
         }
218e99
     }
218e99
@@ -587,8 +612,7 @@ static int qcow2_open(BlockDriverState *bs, QDict *options, int flags,
218e99
     opts = qemu_opts_create_nofail(&qcow2_runtime_opts);
218e99
     qemu_opts_absorb_qdict(opts, options, &local_err);
218e99
     if (error_is_set(&local_err)) {
218e99
-        qerror_report_err(local_err);
218e99
-        error_free(local_err);
218e99
+        error_propagate(errp, local_err);
218e99
         ret = -EINVAL;
218e99
         goto fail;
218e99
     }
218e99
@@ -609,8 +633,8 @@ static int qcow2_open(BlockDriverState *bs, QDict *options, int flags,
218e99
     qemu_opts_del(opts);
218e99
 
218e99
     if (s->use_lazy_refcounts && s->qcow_version < 3) {
218e99
-        qerror_report(ERROR_CLASS_GENERIC_ERROR, "Lazy refcounts require "
218e99
-            "a qcow2 image with at least qemu 1.1 compatibility level");
218e99
+        error_setg(errp, "Lazy refcounts require a qcow2 image with at least "
218e99
+                   "qemu 1.1 compatibility level");
218e99
         ret = -EINVAL;
218e99
         goto fail;
218e99
     }
218e99
@@ -1333,7 +1357,8 @@ static int preallocate(BlockDriverState *bs)
218e99
 static int qcow2_create2(const char *filename, int64_t total_size,
218e99
                          const char *backing_file, const char *backing_format,
218e99
                          int flags, size_t cluster_size, int prealloc,
218e99
-                         QEMUOptionParameter *options, int version)
218e99
+                         QEMUOptionParameter *options, int version,
218e99
+                         Error **errp)
218e99
 {
218e99
     /* Calculate cluster_bits */
218e99
     int cluster_bits;
218e99
@@ -1341,9 +1366,8 @@ static int qcow2_create2(const char *filename, int64_t total_size,
218e99
     if (cluster_bits < MIN_CLUSTER_BITS || cluster_bits > MAX_CLUSTER_BITS ||
218e99
         (1 << cluster_bits) != cluster_size)
218e99
     {
218e99
-        error_report(
218e99
-            "Cluster size must be a power of two between %d and %dk",
218e99
-            1 << MIN_CLUSTER_BITS, 1 << (MAX_CLUSTER_BITS - 10));
218e99
+        error_setg(errp, "Cluster size must be a power of two between %d and "
218e99
+                   "%dk", 1 << MIN_CLUSTER_BITS, 1 << (MAX_CLUSTER_BITS - 10));
218e99
         return -EINVAL;
218e99
     }
218e99
 
218e99
@@ -1362,15 +1386,18 @@ static int qcow2_create2(const char *filename, int64_t total_size,
218e99
     BlockDriverState* bs;
218e99
     QCowHeader header;
218e99
     uint8_t* refcount_table;
218e99
+    Error *local_err = NULL;
218e99
     int ret;
218e99
 
218e99
-    ret = bdrv_create_file(filename, options, NULL);
218e99
+    ret = bdrv_create_file(filename, options, &local_err);
218e99
     if (ret < 0) {
218e99
+        error_propagate(errp, local_err);
218e99
         return ret;
218e99
     }
218e99
 
218e99
-    ret = bdrv_file_open(&bs, filename, NULL, BDRV_O_RDWR, NULL);
218e99
+    ret = bdrv_file_open(&bs, filename, NULL, BDRV_O_RDWR, &local_err);
218e99
     if (ret < 0) {
218e99
+        error_propagate(errp, local_err);
218e99
         return ret;
218e99
     }
218e99
 
218e99
@@ -1400,6 +1427,7 @@ static int qcow2_create2(const char *filename, int64_t total_size,
218e99
 
218e99
     ret = bdrv_pwrite(bs, 0, &header, sizeof(header));
218e99
     if (ret < 0) {
218e99
+        error_setg_errno(errp, -ret, "Could not write qcow2 header");
218e99
         goto out;
218e99
     }
218e99
 
218e99
@@ -1409,6 +1437,7 @@ static int qcow2_create2(const char *filename, int64_t total_size,
218e99
     g_free(refcount_table);
218e99
 
218e99
     if (ret < 0) {
218e99
+        error_setg_errno(errp, -ret, "Could not write refcount table");
218e99
         goto out;
218e99
     }
218e99
 
218e99
@@ -1422,13 +1451,16 @@ static int qcow2_create2(const char *filename, int64_t total_size,
218e99
     BlockDriver* drv = bdrv_find_format("qcow2");
218e99
     assert(drv != NULL);
218e99
     ret = bdrv_open(bs, filename, NULL,
218e99
-        BDRV_O_RDWR | BDRV_O_CACHE_WB | BDRV_O_NO_FLUSH, drv, NULL);
218e99
+        BDRV_O_RDWR | BDRV_O_CACHE_WB | BDRV_O_NO_FLUSH, drv, &local_err);
218e99
     if (ret < 0) {
218e99
+        error_propagate(errp, local_err);
218e99
         goto out;
218e99
     }
218e99
 
218e99
     ret = qcow2_alloc_clusters(bs, 2 * cluster_size);
218e99
     if (ret < 0) {
218e99
+        error_setg_errno(errp, -ret, "Could not allocate clusters for qcow2 "
218e99
+                         "header and refcount table");
218e99
         goto out;
218e99
 
218e99
     } else if (ret != 0) {
218e99
@@ -1439,6 +1471,7 @@ static int qcow2_create2(const char *filename, int64_t total_size,
218e99
     /* Okay, now that we have a valid image, let's give it the right size */
218e99
     ret = bdrv_truncate(bs, total_size * BDRV_SECTOR_SIZE);
218e99
     if (ret < 0) {
218e99
+        error_setg_errno(errp, -ret, "Could not resize image");
218e99
         goto out;
218e99
     }
218e99
 
218e99
@@ -1446,6 +1479,8 @@ static int qcow2_create2(const char *filename, int64_t total_size,
218e99
     if (backing_file) {
218e99
         ret = bdrv_change_backing_file(bs, backing_file, backing_format);
218e99
         if (ret < 0) {
218e99
+            error_setg_errno(errp, -ret, "Could not assign backing file '%s' "
218e99
+                             "with format '%s'", backing_file, backing_format);
218e99
             goto out;
218e99
         }
218e99
     }
218e99
@@ -1457,6 +1492,7 @@ static int qcow2_create2(const char *filename, int64_t total_size,
218e99
         ret = preallocate(bs);
218e99
         qemu_co_mutex_unlock(&s->lock);
218e99
         if (ret < 0) {
218e99
+            error_setg_errno(errp, -ret, "Could not preallocate metadata");
218e99
             goto out;
218e99
         }
218e99
     }
218e99
@@ -1477,6 +1513,8 @@ static int qcow2_create(const char *filename, QEMUOptionParameter *options,
218e99
     size_t cluster_size = DEFAULT_CLUSTER_SIZE;
218e99
     int prealloc = 0;
218e99
     int version = 2;
218e99
+    Error *local_err = NULL;
218e99
+    int ret;
218e99
 
218e99
     /* Read out options */
218e99
     while (options && options->name) {
218e99
@@ -1498,8 +1536,8 @@ static int qcow2_create(const char *filename, QEMUOptionParameter *options,
218e99
             } else if (!strcmp(options->value.s, "metadata")) {
218e99
                 prealloc = 1;
218e99
             } else {
218e99
-                fprintf(stderr, "Invalid preallocation mode: '%s'\n",
218e99
-                    options->value.s);
218e99
+                error_setg(errp, "Invalid preallocation mode: '%s'",
218e99
+                           options->value.s);
218e99
                 return -EINVAL;
218e99
             }
218e99
         } else if (!strcmp(options->name, BLOCK_OPT_COMPAT_LEVEL)) {
218e99
@@ -1508,8 +1546,8 @@ static int qcow2_create(const char *filename, QEMUOptionParameter *options,
218e99
             } else if (!strcmp(options->value.s, "1.1")) {
218e99
                 version = 3;
218e99
             } else {
218e99
-                fprintf(stderr, "Invalid compatibility level: '%s'\n",
218e99
-                    options->value.s);
218e99
+                error_setg(errp, "Invalid compatibility level: '%s'",
218e99
+                           options->value.s);
218e99
                 return -EINVAL;
218e99
             }
218e99
         } else if (!strcmp(options->name, BLOCK_OPT_LAZY_REFCOUNTS)) {
218e99
@@ -1519,19 +1557,23 @@ static int qcow2_create(const char *filename, QEMUOptionParameter *options,
218e99
     }
218e99
 
218e99
     if (backing_file && prealloc) {
218e99
-        fprintf(stderr, "Backing file and preallocation cannot be used at "
218e99
-            "the same time\n");
218e99
+        error_setg(errp, "Backing file and preallocation cannot be used at "
218e99
+                   "the same time");
218e99
         return -EINVAL;
218e99
     }
218e99
 
218e99
     if (version < 3 && (flags & BLOCK_FLAG_LAZY_REFCOUNTS)) {
218e99
-        fprintf(stderr, "Lazy refcounts only supported with compatibility "
218e99
-                "level 1.1 and above (use compat=1.1 or greater)\n");
218e99
+        error_setg(errp, "Lazy refcounts only supported with compatibility "
218e99
+                   "level 1.1 and above (use compat=1.1 or greater)");
218e99
         return -EINVAL;
218e99
     }
218e99
 
218e99
-    return qcow2_create2(filename, sectors, backing_file, backing_fmt, flags,
218e99
-                         cluster_size, prealloc, options, version);
218e99
+    ret = qcow2_create2(filename, sectors, backing_file, backing_fmt, flags,
218e99
+                        cluster_size, prealloc, options, version, &local_err);
218e99
+    if (error_is_set(&local_err)) {
218e99
+        error_propagate(errp, local_err);
218e99
+    }
218e99
+    return ret;
218e99
 }
218e99
 
218e99
 static int qcow2_make_empty(BlockDriverState *bs)
218e99
-- 
218e99
1.7.1
218e99