[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#472233: marked as done (openssh-server: bashism in /bin/sh maintainer script)



Your message dated Sun, 23 Mar 2008 09:52:51 +0000
with message-id <20080323095251.GS16526@riva.ucam.org>
and subject line Re: Bug#472233: openssh-server: bashism in /bin/sh maintainer script
has caused the Debian Bug report #472233,
regarding openssh-server: bashism in /bin/sh maintainer script
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact owner@bugs.debian.org
immediately.)


-- 
472233: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=472233
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message ---
Package: openssh-server
Severity: normal
Version: 1:4.7p1-4
User: debian-release@lists.debian.org
Usertags: goal-dash

Hello maintainer,

While checking the list of packages reported by lintian at[0] I've confirmed 
with the latest checkbashisms (from the 'devscripts' package) that one or 
more maintainer scripts make use of a bashism.

checkbashisms' output:
> possible bashism in ./config line 65 (echo -e):
>   echo -en '\0\0' | 3<&0 sh -c \
> possible bashism in ./preinst line 278 (echo -e):
>     echo -en '\0\0' | 3<&0 sh -c \

Not using bash (or a Debian Policy conformant shell interpreter which doesn't 
provide such an extra feature) as /bin/sh is likely to lead to errors or 
unexpected behaviours.
Please be aware that although bash is currently the default /bin/sh there's a 
release goal for Lenny to make dash the default /bin/sh[1].

If you want more information about dash as /bin/sh, you can read:
http://lists.debian.org/debian-release/2008/01/msg00189.html
https://wiki.ubuntu.com/DashAsBinSh

For more information please refer to the Debian Policy section 10.4 supporting 
this argument at:
http://www.debian.org/doc/debian-policy/ch-files.html#s-scripts

HINT: sometimes these bugs are already fixed in Ubuntu, look at the PTS

[0]http://lintian.debian.org/reports/tags/possible-bashism-in-maintainer-script.html
[1]http://release.debian.org/lenny/goals.txt

Thank you,
-- 
Atomo64 - Raphael

Please avoid sending me Word, PowerPoint or Excel attachments.
See http://www.gnu.org/philosophy/no-word-attachments.html



--- End Message ---
--- Begin Message ---
Source: openssh
Source-Version: 1:4.7p1-5

On Sat, Mar 22, 2008 at 02:18:14PM -0600, Raphael Geissert wrote:
> While checking the list of packages reported by lintian at[0] I've confirmed 
> with the latest checkbashisms (from the 'devscripts' package) that one or 
> more maintainer scripts make use of a bashism.
> 
> checkbashisms' output:
> > possible bashism in ./config line 65 (echo -e):
> >   echo -en '\0\0' | 3<&0 sh -c \
> > possible bashism in ./preinst line 278 (echo -e):
> >     echo -en '\0\0' | 3<&0 sh -c \

Coincidentally, I fixed this yesterday:

openssh (1:4.7p1-5) unstable; urgency=low

  * Recommends: xauth rather than Suggests: xbase-clients.
  * Document in ssh(1) that '-S none' disables connection sharing
    (closes: #471437).
  * Patch from Red Hat / Fedora:
    - SECURITY: Don't use X11 forwarding port which can't be bound on all
      address families, preventing hijacking of X11 forwarding by
      unprivileged users when both IPv4 and IPv6 are configured (closes:
      #463011).
  * Use printf rather than echo -en (a bashism) in openssh-server.config and
    openssh-server.preinst.
  * debconf template translations:
    - Update Finnish (thanks, Esko Arajärvi; closes: #468563).

 -- Colin Watson <cjwatson@debian.org>  Sat, 22 Mar 2008 12:37:00 +0000

-- 
Colin Watson                                       [cjwatson@debian.org]


--- End Message ---

Reply to: