On Thu, Oct 16, 2008 at 10:07:13AM +0200, Arthur de Jong wrote:
> Subject: update-notifier: refuses to start if the admin group exists
> Package: update-notifier
> Version: 0.70.7.debian-4
> Severity: normal
> 
> On my system update-notifier refuses to start with the message:
> 
> * (update-notifier:12663): WARNING **: not starting because user is not in 
> admin group
> 
> In our environment we have an admin group that is used for
> administrative staff (not system administrators).
> 
> It took me quite a while to figure out what was going on and how to fix
> it (not documented in /usr/share/doc or the manual page). It turns out
> you should use the --force option in this situation.
> 
> Please don't use a normal group name that is not a default system group
> (perhaps the adm or operator group should be used instead if it contains
> any members) or provide some other means for administrators to hide
> update-notifier for users that shouldn't be made aware of updates.
How about the following patch? 

=== modified file 'src/update-notifier.c'
--- src/update-notifier.c       2010-04-18 16:01:56 +0000
+++ src/update-notifier.c       2010-05-21 09:53:49 +0000
@@ -455,6 +455,11 @@
    if(grp == NULL) 
       return TRUE;
 
+   // The admin group is not a system group, so we just
+   // treat it as non-existent and return TRUE.
+   if(grp->gr_gid >= 1000)
+      return TRUE;
+
    ng = getgroups (0, NULL);
    groups = (gid_t *) malloc (ng * sizeof (gid_t));


-- 
Julian Andres Klode  - Debian Developer, Ubuntu Member

See http://wiki.debian.org/JulianAndresKlode and http://jak-linux.org/.

Attachment: pgpONSN3wZEdJ.pgp
Description: PGP signature

Reply via email to