Description of problem:
src/pk-transaction-extra.c:pk_transaction_extra_check_library_restart categorizes UIDs as system/session using a hard-coded value 500. Please use UID_MIN from /etc/login.defs for this; we plan to change the boundary.
Please let me know if you want me to prepare a patch.
Version-Release number of selected component (if applicable):
Created attachment 510191 [details]
Code to parse /etc/login.defs
This demonstrates the proper way to parse /etc/login.defs .
Author: Richard Hughes <firstname.lastname@example.org>
Date: Tue Jun 28 14:52:16 2011 +0100
Do not hardcode the UID_MIN as 500. Fixes rh#717110
There's a new upstream release in 6 days, so I'll include this fix in F15 and rawhide then. Thanks for pointing out the problem.
Thanks a lot; could you make sure commit 5456681c29753439766700f5dc431a4a1cf72a2e will get into Fedora 16 as well, please?
system accounts are the ones below SYS_UID_MAX not below UID_MIN. UID_MIN is only the minimum that useradd uses, external user accounts (ldap) can still be below UID_MIN.
(In reply to comment #4)
> system accounts are the ones below SYS_UID_MAX not below UID_MIN. UID_MIN is
> only the minimum that useradd uses, external user accounts (ldap) can still be
> below UID_MIN.
Similarly, external system accounts can be outside of the SYS_* range. PackageKit considers each account to be either a system or user account, there is no "uknown" category.
(In reply to comment #3)
> Thanks a lot; could you make sure commit
> 5456681c29753439766700f5dc431a4a1cf72a2e will get into Fedora 16 as well,
Commit is included in rawhide PackageKit-0.6.17-2, so rawhide should be fine. Thanks!
This bug appears to have been reported against 'rawhide' during the Fedora 19 development cycle.
Changing version to '19'.
(As we did not run this process for some time, it could affect also pre-Fedora 19 development
cycle bugs. We are very sorry. It will help us with cleanup during Fedora 19 End Of Life. Thank you.)
More information and reason for this action is here: