qemu-s390x
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [qemu-s390x] [PATCH v2] s390: avoid potential null dereference ins39


From: Peter Maydell
Subject: Re: [qemu-s390x] [PATCH v2] s390: avoid potential null dereference ins390_pcihost_unplug()
Date: Mon, 7 Jan 2019 15:54:21 +0000

On Mon, 7 Jan 2019 at 15:48, Cornelia Huck <address@hidden> wrote:
>
> On Fri, 4 Jan 2019 22:33:51 +0800
> Li Qiang <address@hidden> wrote:
>
> > What do you think of ‘g_assert_not_reached();’. For example:
> >
> > else {
> >      g_assert_not_reached();
> > }
>
> Sounds good. But please return anyway in the unplug case, so that the
> code is fine if asserts have been configured out.

Hopefully that won't cause the compiler to complain about
unreachable code :-)

thanks
-- PMM



reply via email to

[Prev in Thread] Current Thread [Next in Thread]