From: Olaf Hering <olaf@aepfle.de>
To: xen-devel@lists.xensource.com
Subject: [PATCH 03 of 18] tools/blktap: fix access error in img2qcow.c
Date: Fri, 30 Mar 2012 10:25:20 +0200 [thread overview]
Message-ID: <526135e426fa80dfd58c.1333095920@probook.site> (raw)
In-Reply-To: <patchbomb.1333095917@probook.site>
# HG changeset patch
# User Olaf Hering <olaf@aepfle.de>
# Date 1333046228 -7200
# Node ID 526135e426fa80dfd58c272b41cd3f4ab39c62d6
# Parent 5c77f79cc3064b575c1da2b8fb28eec53dd8bfdc
tools/blktap: fix access error in img2qcow.c
img2qcow.c: In function 'print_bytes':
img2qcow.c:72:9: warning: value computed is not used [-Wunused-value]
Increment the pointer, not the char it points to.
Signed-off-by: Olaf Hering <olaf@aepfle.de>
diff -r 5c77f79cc306 -r 526135e426fa tools/blktap/drivers/img2qcow.c
--- a/tools/blktap/drivers/img2qcow.c
+++ b/tools/blktap/drivers/img2qcow.c
@@ -69,7 +69,7 @@ static void print_bytes(void *ptr, int l
DFPRINTF("Buf dump, length %d:\n",length);
for (k = 0; k < length; k++) {
DFPRINTF("%x",*p);
- *p++;
+ p++;
if(k % 16 == 0) DFPRINTF("\n");
else if(k % 2 == 0) DFPRINTF(" ");
}
next prev parent reply other threads:[~2012-03-30 8:25 UTC|newest]
Thread overview: 53+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-03-30 8:25 [PATCH 00 of 18] tools: fix bugs and build errors triggered by -O2 -Wall -Werror Olaf Hering
2012-03-30 8:25 ` [PATCH 01 of 18] tools/blktap: fix access errors in convert_dev_name_to_num Olaf Hering
2012-04-02 13:47 ` Ian Jackson
2012-04-02 14:04 ` Ian Jackson
2012-04-02 14:42 ` Olaf Hering
2012-04-02 14:44 ` Olaf Hering
2012-04-02 15:04 ` Ian Campbell
2012-04-02 15:08 ` Olaf Hering
2012-04-02 15:16 ` Ian Jackson
2012-03-30 8:25 ` [PATCH 02 of 18] tools/blktap: fix params and physical-device parsing Olaf Hering
2012-04-02 14:38 ` Ian Jackson
2012-03-30 8:25 ` Olaf Hering [this message]
2012-03-30 8:25 ` [PATCH 04 of 18] tools/blktap: fix access error in qcow2raw.c Olaf Hering
2012-03-30 8:25 ` [PATCH 05 of 18] tools/blktap2: fix build errors caused by Werror in vhd_journal_write_entry Olaf Hering
2012-04-02 13:55 ` Ian Jackson
2012-03-30 8:25 ` [PATCH 06 of 18] tools/blktap2: fix out of bounds access in block-log.c Olaf Hering
2012-04-02 14:31 ` Ian Jackson
2012-03-30 8:25 ` [PATCH 07 of 18] tools/blktap2: fix build errors caused by Werror in tdqcow_get_parent_id Olaf Hering
2012-04-02 13:52 ` Ian Jackson
2012-03-30 8:25 ` [PATCH 08 of 18] tools/blktap2: fix access error in img2qcow.c Olaf Hering
2012-03-30 8:25 ` [PATCH 09 of 18] tools/blktap2: fix access error in qcow2raw.c Olaf Hering
2012-03-30 8:25 ` [PATCH 10 of 18] tools/libvchan: fix build errors caused by -Werror in node-select.c Olaf Hering
2012-04-02 13:59 ` Ian Jackson
2012-04-02 14:40 ` Olaf Hering
2012-04-02 19:54 ` Olaf Hering
2012-04-03 8:23 ` Ian Campbell
2012-04-03 9:07 ` Olaf Hering
2012-04-03 10:27 ` Ian Jackson
2012-04-03 10:20 ` Ian Jackson
2012-03-30 8:25 ` [PATCH 11 of 18] tools/libxl: fix build errors caused by Werror in disk_eject_xswatch_callback Olaf Hering
2012-04-02 13:53 ` Ian Jackson
2012-04-02 19:58 ` Olaf Hering
2012-04-03 8:30 ` Ian Campbell
2012-04-03 9:11 ` Olaf Hering
2012-04-03 9:24 ` Ian Campbell
2012-04-03 10:28 ` Ian Jackson
2012-04-03 10:21 ` Ian Jackson
2012-03-30 8:25 ` [PATCH 12 of 18] tools/memshr: fix build errors caused by Werror Olaf Hering
2012-04-02 14:39 ` Ian Jackson
2012-03-30 8:25 ` [PATCH 13 of 18] tools/xenpaging: fix build errors caused by -Werror Olaf Hering
2012-04-02 14:01 ` Ian Jackson
2012-03-30 8:25 ` [PATCH 14 of 18] tools/libvchan: fix build errors caused by Werror in io.c Olaf Hering
2012-04-02 14:33 ` Ian Jackson
2012-04-02 14:37 ` Olaf Hering
2012-04-02 14:39 ` Ian Jackson
2012-03-30 8:25 ` [PATCH 15 of 18] tools/blktap2: remove static string table from header file Olaf Hering
2012-04-02 14:32 ` Ian Jackson
2012-03-30 8:25 ` [PATCH 16 of 18] tools/blktap2: fix build errors caused by Werror, remove blkif_op_name Olaf Hering
2012-04-02 14:06 ` Ian Jackson
2012-03-30 8:25 ` [PATCH 17 of 18] tools/blktap2: remove unused labels Olaf Hering
2012-04-02 14:32 ` Ian Jackson
2012-03-30 8:25 ` [PATCH 18 of 18] tools/blktap2: fix build errors caused by Werror, remove unused variables Olaf Hering
2012-03-30 12:44 ` Olaf Hering
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=526135e426fa80dfd58c.1333095920@probook.site \
--to=olaf@aepfle.de \
--cc=xen-devel@lists.xensource.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.