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 ] ; then"....or
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.


Attachment: signature.asc
Description: Digital signature

Reply via email to