Package: john
Version: 1.6-37

The postinst contains this snippet:

       tmp=`tempfile`
       if [ "$INSTCRON" = "true" ] && [ -f $cronfile ]; then
               cat $cronfile | sed -e 's/^#00/00/' >$tmp
               mv $tmp $cronfile
       else
       # We have two options here, leave the file as it is (the user
       # might have modified it) or revert the previous change. I'm
       # opting for the second case to make the debconf operation
       # idempotent (jfs)
               if [ -f $cronfile ] ; then
                       cat $cronfile | sed -e 's/^00/#00/' >$tmp
                       mv $tmp $cronfile
               fi
        fi

There is a possible code path (which seems to be excercised by piuparts)
which results in the temporary file not being removed:

        0m7.7s ERROR: Package purging left files on system:
          /tmp/fileH2Sg7E
        
Adding an "else" ot the innermost if to remove $tmp should fix this.

-- 
C is the *wrong* language for your application.


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to