We're checking whether a user in the wtmp database should be excluded, but we use the minimal UID instead of the entry's UID to check for addition.
Created attachment 58110 [details] [review] Fix incorrect argument to is_excluded() We're checking whether a user in the wtmp database should be excluded, but we use the minimal UID instead of the entry's UID to check for addition.
Comment on attachment 58110 [details] [review] Fix incorrect argument to is_excluded() Review of attachment 58110 [details] [review]: ----------------------------------------------------------------- ::: src/daemon.c @@ +198,4 @@ > > if (daemon_local_user_is_excluded (daemon, > wtmp_entry->ut_user, > + wtmp_entry->ut_pid)) { that says pid not uid. You'll probably have to do getpwnam(wtmp_entry->ut_user)
This got fixed by bug 44408
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.