On Tue, Nov 17, 2020 at 05:30:44PM +0100, Kevin Wolf wrote: > If the 'service' property is not set, we'll call pam_start() with a NULL > pointer for the service name. This fails and leaves a message like this > in the syslog: > > qemu-storage-daemon[294015]: PAM pam_start: invalid argument: service == NULL > > Make specifying the property mandatory and catch the error already > during the creation of the object. > > Signed-off-by: Kevin Wolf <kw...@redhat.com> > --- > authz/pamacct.c | 6 ++++++ > 1 file changed, 6 insertions(+)
Reviewed-by: Daniel P. Berrangé <berra...@redhat.com> 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 :|
