[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3] error: passing a negative value to an os_err
From: |
SeokYeon Hwang |
Subject: |
Re: [Qemu-devel] [PATCH v3] error: passing a negative value to an os_errno is wrong |
Date: |
Wed, 12 Nov 2014 16:00:11 +0900 |
> -----Original Message-----
> From: Eric Blake [mailto:address@hidden
> Sent: Tuesday, November 11, 2014 1:14 AM
> To: SeokYeon Hwang; address@hidden
> Cc: address@hidden; address@hidden; address@hidden;
> address@hidden
> Subject: Re: [PATCH v3] error: passing a negative value to an os_errno is
> wrong
>
> On 11/09/2014 10:54 PM, SeokYeon Hwang wrote:
> > Added 'assert(os_errno > 0)' in 'error_set_errno()'.
>
> Commit message...
>
> > Fixed errno since it passes wrong value to 'error_set_errno()'.
> >
> > Signed-off-by: SeokYeon Hwang <address@hidden>
> > ---
> > hw/pci/pcie.c | 2 +-
>
> > +++ 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);
>
> ...does not match code. (Code is correct).
>
> --
> Eric Blake eblake redhat com +1-919-301-3266
> Libvirt virtualization library http://libvirt.org
You're right. Thank you.