I've verified that Brice Figureau's bug is valid, and that his patch
fixes it. Attached is a patch complete with an updated debian/changelog
as well. I will NMU it shortly (since we're in a 0-day NMU period) and
release managers to accept the change for sarge.

diff -ruN old/debian/changelog logrotate-3.7/debian/changelog
--- old/debian/changelog	2005-05-15 23:50:43.000000000 +0300
+++ logrotate-3.7/debian/changelog	2005-05-16 00:43:15.000000000 +0300
@@ -1,3 +1,12 @@
+logrotate (3.7-2.1) unstable; urgency=high
+
+  * Non-maintainer upload.
+  * When calling execlp to run scripts, give execlp all the arguments it
+    needs. This way, it doesn't eat the first actual command line 
+    argument (argv[1]). Fix by Brice Figureau. Closes: #276172.
+
+ -- Lars Wirzenius <[EMAIL PROTECTED]>  Mon, 16 May 2005 00:40:00 +0300
+
 logrotate (3.7-2) unstable; urgency=high
 
   * Added commented out stuff in debian/rules to build a 
diff -ruN old/logrotate.c logrotate-3.7/logrotate.c
--- old/logrotate.c	2004-02-05 20:42:13.000000000 +0200
+++ logrotate-3.7/logrotate.c	2005-05-16 00:40:14.000000000 +0300
@@ -110,7 +110,7 @@
     close(fd);
 
     if (!fork()) {
-	execlp(filespec, logfn, NULL);
+	execlp(filespec, filespec, logfn, NULL);
 	exit(1);
     }
 

Reply via email to