NetBSD Problem Report #41544

From www@NetBSD.org  Fri Jun  5 09:06:28 2009
Return-Path: <www@NetBSD.org>
Received: from mail.netbsd.org (mail.netbsd.org [204.152.190.11])
	by www.NetBSD.org (Postfix) with ESMTP id 9432063B88A
	for <gnats-bugs@gnats.netbsd.org>; Fri,  5 Jun 2009 09:06:28 +0000 (UTC)
Message-Id: <20090605090628.531B263B11D@www.NetBSD.org>
Date: Fri,  5 Jun 2009 09:06:28 +0000 (UTC)
From: netbsd@eq.cz
Reply-To: netbsd@eq.cz
To: gnats-bugs@NetBSD.org
Subject: /etc/weekly changes permissions of /var/db/locate.database
X-Send-Pr-Version: www-1.0

>Number:         41544
>Category:       misc
>Synopsis:       /etc/weekly changes permissions of /var/db/locate.database
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    misc-bug-people
>State:          closed
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Fri Jun 05 09:10:00 +0000 2009
>Closed-Date:    Sat Jan 30 19:22:33 +0000 2010
>Last-Modified:  Sat Jan 30 19:25:01 +0000 2010
>Originator:     rudolf
>Release:        NetBSD 5.0_STABLE
>Organization:
>Environment:
NetBSD 5.0_STABLE (GENERIC) #0: Tue Jun  2 21:21:05 UTC 2009  builds@b3.netbsd.org:/home/builds/ab/netbsd-5/i386/200906020000Z-obj/home/builds/ab/netbsd-5/src/sys/arch/i386/compile/GENERIC i386
>Description:
The /etc/weekly script does not restore permissions of /var/db/locate.database back to 0664, cf.:
http://nxr.netbsd.org/source/xref/etc/weekly#69
http://nxr.netbsd.org/source/xref/etc/Makefile#303


>How-To-Repeat:
Install the system, the permissions of /var/db/locate.database are 0664, run /etc/weekly, the permissions are 0644.
>Fix:
--- etc/weekly.orig     2009-06-05 10:32:48.000000000 +0200
+++ etc/weekly  2009-06-05 10:32:48.000000000 +0200
@@ -74,6 +74,7 @@
                chown nobody:nobody /var/db/locate.database
                (cd / && nice -5 su -m nobody -c /usr/libexec/locate.updatedb 2>/dev/null)
                chown root:wheel /var/db/locate.database
+               chmod 664 /var/db/locate.database
        else
                echo "Not rebuilding locate database; no /var/db/locate.database"
        fi

>Release-Note:

>Audit-Trail:
From: David Holland <dholland-bugs@netbsd.org>
To: gnats-bugs@NetBSD.org
Cc: 
Subject: Re: misc/41544: /etc/weekly changes permissions of
	/var/db/locate.database
Date: Sat, 6 Jun 2009 20:21:35 +0000

 On Fri, Jun 05, 2009 at 09:10:01AM +0000, netbsd@eq.cz wrote:
  > The /etc/weekly script does not restore permissions of /var/db/locate.database back to 0664, cf.:
  > http://nxr.netbsd.org/source/xref/etc/weekly#69
  > http://nxr.netbsd.org/source/xref/etc/Makefile#303

 I don't see any reason it should be mode 664; it looks instead like
 etc/Makefile is outdated.

 --- etc/Makefile	23 Apr 2009 11:29:29 -0000	1.366
 +++ etc/Makefile	6 Jun 2009 20:19:37 -0000
 @@ -304,7 +304,7 @@ install-etc-files: .PHONY .MAKE check_DE
  		${BINOWN} operator	664	/etc/dumpdates  \
  		${BINOWN} operator	600	/etc/skeykeys \
  		root wheel		600	/var/at/at.deny \
 -		nobody ${BINGRP}	664	/var/db/locate.database \
 +		root wheel		644	/var/db/locate.database \
  		${BINOWN} ${BINGRP}	600	/var/log/authlog \
  		root wheel		600	/var/log/cron \
  		${BINOWN} ${UTMPGRP}	664	/var/log/lastlog \

 Anyone else have an opinion?

 -- 
 David A. Holland
 dholland@netbsd.org

From: "David A. Holland" <dholland@netbsd.org>
To: gnats-bugs@gnats.NetBSD.org
Cc: 
Subject: PR/41544 CVS commit: src/etc
Date: Sun, 10 Jan 2010 06:13:25 +0000

 Module Name:	src
 Committed By:	dholland
 Date:		Sun Jan 10 06:13:25 UTC 2010

 Modified Files:
 	src/etc: Makefile

 Log Message:
 Fix installation permissions of /var/db/locate.database as per PR misc/41544.


 To generate a diff of this commit:
 cvs rdiff -u -r1.376 -r1.377 src/etc/Makefile

 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.

From: rudolf <netbsd@eq.cz>
To: gnats-bugs@NetBSD.org
Cc: 
Subject: Re: PR/41544 CVS commit: src/etc
Date: Sun, 10 Jan 2010 10:40:37 +0100

 Hi,

 David A. Holland wrote:
 > The following reply was made to PR misc/41544; it has been noted by GNATS.
 > 
 > From: "David A. Holland" <dholland@netbsd.org>
 > To: gnats-bugs@gnats.NetBSD.org
 > Cc: 
 > Subject: PR/41544 CVS commit: src/etc
 > Date: Sun, 10 Jan 2010 06:13:25 +0000
 > 
 >  Module Name:	src
 >  Committed By:	dholland
 >  Date:		Sun Jan 10 06:13:25 UTC 2010
 >  
 >  Modified Files:
 >  	src/etc: Makefile
 >  
 >  Log Message:
 >  Fix installation permissions of /var/db/locate.database as per PR misc/41544.

 IIUC you decided that "664" permissions are right (you changed only the 
 owner and group). In that case we still need the change to /etc/weekly 
 mentioned in the PR if we want to have the same state of permissions and 
 ownership "before /etc/weekly" and after it.

 (But probably you wanted to change the permissions in src/etc/Makefile 
 to 644?)

 r.

From: David Holland <dholland-bugs@netbsd.org>
To: gnats-bugs@NetBSD.org
Cc: misc-bug-people@netbsd.org, gnats-admin@netbsd.org,
	netbsd-bugs@netbsd.org, netbsd@eq.cz
Subject: Re: PR/41544 CVS commit: src/etc
Date: Tue, 12 Jan 2010 04:42:15 +0000

 On Sun, Jan 10, 2010 at 09:45:01AM +0000, rudolf wrote:
  >  >  Log Message:
  >  >  Fix installation permissions of /var/db/locate.database as per PR misc/41544.
  >  
  >  IIUC you decided that "664" permissions are right (you changed only the 
  >  owner and group). In that case we still need the change to /etc/weekly 
  >  mentioned in the PR if we want to have the same state of permissions and 
  >  ownership "before /etc/weekly" and after it.
  >  
  >  (But probably you wanted to change the permissions in src/etc/Makefile 
  >  to 644?)

 No, you're right, I messed up. Bah. :-/

 -- 
 David A. Holland
 dholland@netbsd.org

From: "David A. Holland" <dholland@netbsd.org>
To: gnats-bugs@gnats.NetBSD.org
Cc: 
Subject: PR/41544 CVS commit: src/etc
Date: Tue, 12 Jan 2010 04:44:06 +0000

 Module Name:	src
 Committed By:	dholland
 Date:		Tue Jan 12 04:44:06 UTC 2010

 Modified Files:
 	src/etc: Makefile

 Log Message:
 Fix previous: use correct mode as well as owner/group.
 My bad. PR misc/41544.


 To generate a diff of this commit:
 cvs rdiff -u -r1.377 -r1.378 src/etc/Makefile

 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.

State-Changed-From-To: open->pending-pullups
State-Changed-By: dholland@NetBSD.org
State-Changed-When: Sun, 24 Jan 2010 19:31:43 +0000
State-Changed-Why:
pullup-5 #1273


State-Changed-From-To: pending-pullups->closed
State-Changed-By: snj@NetBSD.org
State-Changed-When: Sat, 30 Jan 2010 19:22:33 +0000
State-Changed-Why:
Pulled up.


From: Soren Jacobsen <snj@netbsd.org>
To: gnats-bugs@gnats.NetBSD.org
Cc: 
Subject: PR/41544 CVS commit: [netbsd-5] src/etc
Date: Sat, 30 Jan 2010 19:21:15 +0000

 Module Name:	src
 Committed By:	snj
 Date:		Sat Jan 30 19:21:15 UTC 2010

 Modified Files:
 	src/etc [netbsd-5]: Makefile

 Log Message:
 Pull up following revision(s) (requested by dholland in ticket #1273):
 	etc/Makefile: revision 1.377
 	etc/Makefile: revision 1.378
 Fix installation permissions of /var/db/locate.database as per PR misc/41544.


 To generate a diff of this commit:
 cvs rdiff -u -r1.359.2.5 -r1.359.2.6 src/etc/Makefile

 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.

>Unformatted:

Home	PR Database Search