Bug#393300: preinst script bug

2006-10-16 Thread Christian Perrier
Quoting H. S. Teoh ([EMAIL PROTECTED]):
 Package: console-common
 Version: 0.7.65
 Severity: normal
 
 Here's the output from dpkg during the latest upgrade to 0.7.65:
 
 Preconfiguring packages ...
 (Reading database ... 27513 files and directories currently installed.)
 Preparing to replace console-common 0.7.64 (using 
 .../console-common_0.7.65_all.deb) ...
 /var/lib/dpkg/tmp.ci/preinst: line 63: [: : integer expression expected
 /var/lib/dpkg/tmp.ci/preinst: line 63: [: : integer expression expected
 Unpacking replacement console-common ...
 
 
 Looks like unsafe/buggy shell code in preinst.


Yep, definitely..:-(. I fixed that bug too quickly. Quoting the string
is correct but I didn't pay attention to the -eq which requires
integers.

Code that would better work is if [ $errors = 1 ] ; thenor
just initialize the $errors variable to 0. Alastair, comments? I was
expecting some about the former bug and I finally decided to upload
just to be sure to missing the freeze deadline.




signature.asc
Description: Digital signature


Bug#393300: preinst script bug

2006-10-15 Thread H. S. Teoh
Package: console-common
Version: 0.7.65
Severity: normal

Here's the output from dpkg during the latest upgrade to 0.7.65:

Preconfiguring packages ...
(Reading database ... 27513 files and directories currently installed.)
Preparing to replace console-common 0.7.64 (using 
.../console-common_0.7.65_all.deb) ...
/var/lib/dpkg/tmp.ci/preinst: line 63: [: : integer expression expected
/var/lib/dpkg/tmp.ci/preinst: line 63: [: : integer expression expected
Unpacking replacement console-common ...


Looks like unsafe/buggy shell code in preinst.


T

-- 
Those who have not appreciated the beauty of language are not qualified
to bemoan its flaws.


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