Package: tiger
Version: 1:3.2.2-11
Severity: minor
Tags: patch

Hi,

the first part of the attached patch fixes an issue with deleting multiple 
files in the delete() function of initdefs. When deleting more than one file 
and e.g. the first file doesn't exist, all the rest files aren't deleted as 
well, because the variable goahead isn't reseted within the for loop.

The second part fixes an issue in the find_file script. The find_file
script creates the file mount.list$$ but doesn't delete it on exit.
Otherwise it seems that the mount.list$$ file is never used so do we have 
to create it?

Hannes
diff -Naur tiger-3.2.2.orig/initdefs tiger-3.2.2/initdefs
--- tiger-3.2.2.orig/initdefs	2009-09-02 10:44:43.000000000 +0200
+++ tiger-3.2.2/initdefs	2009-09-02 11:01:49.000000000 +0200
@@ -115,13 +115,13 @@
 #
 delete()
 {
-  goahead=1
   if [ -z "$RM" ] ; then
   	echo "--ERROR-- [init001e] The variable RM is not defined. Aborting."
 	exit 1
   fi
   for __file
   do
+    goahead=1
     if [ -n "$LS" ]
     then 
     	if [ -z "`$LS \"$__file\" 2>/dev/null`" ] 
diff -Naur tiger-3.2.2.orig/scripts/find_files tiger-3.2.2/scripts/find_files
--- tiger-3.2.2.orig/scripts/find_files	2009-09-02 10:44:43.000000000 +0200
+++ tiger-3.2.2/scripts/find_files	2009-09-02 11:05:13.000000000 +0200
@@ -392,7 +392,7 @@
   [ -s "$file" ] && $CAT $file
 done
 
-delete $pidlist ${outfile}*
+delete $pidlist $mountlist ${outfile}*
 delete $WORKDIR/suid.msgs.$$ $WORKDIR/sgid.msgs.$$ \
        $WORKDIR/other.msgs.$$ $WORKDIR/devs.msgs.$$ \
       $WORKDIR/link.msgs.$$ $WORKDIR/wdir.msgs.$$ $WORKDIR/noowner.msgs.$$

Reply via email to