[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH for-5.2? 0/2] authz: Add missing NULL checks
From: |
Daniel P . Berrangé |
Subject: |
Re: [PATCH for-5.2? 0/2] authz: Add missing NULL checks |
Date: |
Tue, 17 Nov 2020 16:44:43 +0000 |
User-agent: |
Mutt/1.14.6 (2020-07-11) |
On Tue, Nov 17, 2020 at 05:30:43PM +0100, Kevin Wolf wrote:
> While trying to write a QAPI schema for user creatable object types, I
> have to figure out whether properties are mandatory or options.
>
> Turns out that some authz object types have properties that should be
> mandatory because the code assumes they are non-NULL, but we never check
> that they are actually given.
Hmm, avoiding manual code to check for mandatory options will be a
nice plus point of using QAPI
Regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|