[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v4] error: passing a negative value to an os_err
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH v4] error: passing a negative value to an os_errno is wrong |
Date: |
Wed, 12 Nov 2014 08:35:25 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.3 (gnu/linux) |
SeokYeon Hwang <address@hidden> writes:
> Added 'assert(os_errno >= 0)' in 'error_set_errno()'.
>
> Signed-off-by: SeokYeon Hwang <address@hidden>
> ---
> util/error.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/util/error.c b/util/error.c
> index 2ace0d8..6c9d995 100644
> --- a/util/error.c
> +++ b/util/error.c
> @@ -62,6 +62,7 @@ void error_set_errno(Error **errp, int os_errno, ErrorClass
> err_class,
> return;
> }
> assert(*errp == NULL);
> + assert(os_errno >= 0);
>
> err = g_malloc0(sizeof(*err));
Too risky for 2.2 at this stage, but fine for 2.3.
Reviewed-by: Markus Armbruster <address@hidden>