]> xenbits.xensource.com Git - qemu-xen.git/commitdiff
qemu-img: remove unused variable
authorMiroslav Rezanina <mrezanin@redhat.com>
Wed, 9 Nov 2022 15:57:13 +0000 (10:57 -0500)
committerAnthony PERARD <anthony.perard@citrix.com>
Mon, 17 Jul 2023 11:05:26 +0000 (12:05 +0100)
Variable block_count used in img_dd function is only incremented but never read.
This causes 'Unused but set variable' warning on Clang 15.0.1 compiler.

Removing the variable to prevent the warning.

Signed-off-by: Miroslav Rezanina <mrezanin@redhat.com>
Reviewed-by: Thomas Huth <thuth@redhat.com>
Message-Id: <e86d5b57f9d13bde995c616a533b876f1fb8a527.1668009030.git.mrezanin@redhat.com>
Signed-off-by: Thomas Huth <thuth@redhat.com>
(cherry picked from commit 0f48c47c679bc29fceb3a67283ee3b78175524bf)

qemu-img.c

index 908fd0cce5b4fc90a4798ea4a1c49e38aa9fbdba..314ca0534d8cf6a34296661b02918b5e8afd5e42 100644 (file)
@@ -4886,7 +4886,7 @@ static int img_dd(int argc, char **argv)
     const char *out_fmt = "raw";
     const char *fmt = NULL;
     int64_t size = 0;
-    int64_t block_count = 0, out_pos, in_pos;
+    int64_t out_pos, in_pos;
     bool force_share = false;
     struct DdInfo dd = {
         .flags = 0,
@@ -5086,7 +5086,7 @@ static int img_dd(int argc, char **argv)
 
     in.buf = g_new(uint8_t, in.bsz);
 
-    for (out_pos = 0; in_pos < size; block_count++) {
+    for (out_pos = 0; in_pos < size; ) {
         int in_ret, out_ret;
 
         if (in_pos + in.bsz > size) {