Re: [Qemu-devel] [PATCH v3 2/2] qemu-io: Fix if scoping bug

2011-07-12 Thread Kevin Wolf
Am 11.07.2011 17:20, schrieb Devin Nakamura:
 Fix a bug caused by lack of braces in if statement
 
 Lack of braces means that if(count  0x1ff) is never reached
 
 Conflicts:
 
   qemu-io.c
 
 Signed-off-by: Devin Nakamura devin...@gmail.com

Thanks, applied both to the block branch.

Kevin



[Qemu-devel] [PATCH v3 2/2] qemu-io: Fix if scoping bug

2011-07-11 Thread Devin Nakamura
Fix a bug caused by lack of braces in if statement

Lack of braces means that if(count  0x1ff) is never reached

Conflicts:

qemu-io.c

Signed-off-by: Devin Nakamura devin...@gmail.com
---
 qemu-io.c |3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/qemu-io.c b/qemu-io.c
index e3c825f..a553d0c 100644
--- a/qemu-io.c
+++ b/qemu-io.c
@@ -449,7 +449,7 @@ static int read_f(int argc, char **argv)
 return 0;
 }
 
-if (!pflag)
+if (!pflag) {
 if (offset  0x1ff) {
 printf(offset % PRId64  is not sector aligned\n,
offset);
@@ -460,6 +460,7 @@ static int read_f(int argc, char **argv)
count);
 return 0;
 }
+}
 
 buf = qemu_io_alloc(count, 0xab);
 
-- 
1.7.6.rc1