[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 2/2] qemu-io: Fix if scoping bug
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH 2/2] qemu-io: Fix if scoping bug |
Date: |
Tue, 14 Jun 2011 10:42:02 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/23.1 (gnu/linux) |
Devin Nakamura <address@hidden> writes:
> Fix a bug caused by lack of braces in if statement
You describe the bug's cause. That's good. Please also describe the
bug's effect, i.e. what exactly is broken for users.
>
> Signed-off-by: Devin Nakamura <address@hidden>
> ---
> qemu-io.c | 4 ++--
> 1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/qemu-io.c b/qemu-io.c
> index 53adb76..1c4f684 100644
> --- a/qemu-io.c
> +++ b/qemu-io.c
> @@ -433,12 +433,12 @@ static int read_f(int argc, char **argv)
> return 0;
> }
>
> - if (!pflag)
> + if (!pflag){
Put a space between ) and {
> if (offset & 0x1ff) {
> printf("offset %" PRId64 " is not sector aligned\n",
> offset);
> return 0;
> -
> + }
> if (count & 0x1ff) {
> printf("count %d is not sector aligned\n",
> count);