Package: findutils
Version: 4.4.1-1
Severity: wishlist
Tags: patch
User: ubuntu-de...@lists.ubuntu.com
Usertags: origin-ubuntu jaunty ubuntu-patch




*** /tmp/tmpJKR5Ck
In Ubuntu, we've applied the attached patch to achieve the following:

  * Merge from debian unstable, remaining changes:
    - debian/locate-cron.daily: Test if called by root (LP: #147911),

We thought you might be interested in doing the same. 

The rational is:

If you call findutils' /etc/cron.daily/find as normal user, you'll get
no error, but a prompt from "su", which gets called. The label text
for the "su" prompt get suppressed, so this is quite confusing, if you
call the script (out of curiosity) with your normal user.

Cheers,
 Michael

-- System Information:
Debian Release: 5.0
  APT prefers jaunty-updates
  APT policy: (500, 'jaunty-updates'), (500, 'jaunty-security'), (500, 
'jaunty-proposed'), (500, 'jaunty-backports'), (500, 'jaunty')
Architecture: i386 (i686)

Kernel: Linux 2.6.28-11-generic (SMP w/2 CPU cores)
Locale: LANG=en_DK.UTF-8, LC_CTYPE=en_DK.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
diff -u findutils-4.4.1/debian/locate-cron.daily findutils-4.4.1/debian/locate-cron.daily
--- findutils-4.4.1/debian/locate-cron.daily
+++ findutils-4.4.1/debian/locate-cron.daily
@@ -11,6 +11,11 @@
 
 [ -e /usr/bin/updatedb.findutils ] || exit 0
 
+if [ "$(id -u)" != "0" ]; then
+	echo "You must be root."
+	exit 1
+fi
+ 
 # Global options for invocations of find(1)
 FINDOPTIONS='-ignore_readdir_race'
 # filesystems which are pruned from updatedb database

Reply via email to