Bug#301233: libpam-mount: a problem with mounts option passing

[Jörg Sommer]

Package: libpam-mount
Version: 0.9.22-2
Severity: normal

Hi,

I have a problem with mount. It passes the options in a strange way to
mount.crypt. It prepends "rw, ". So the option you see in mount.cpryt is
-o "rw, cipher=blowfish". The space makes problems in parsing the
options. This leads to the cipher isn't correct passed to cyrptsetup
which makes it fail to set up the crypt device correctly.

I've added him to the case statment.

-               cipher )
+               " cipher"|cipher )
-               keysize )
+               " keysize"|keysize )
-               hash )
+               " hash"|hash )
-               loop )
+               " loop"|loop )

Jörg.

-- System Information:
Debian Release: 3.1
  APT prefers unstable
  APT policy: (990, 'unstable')
Architecture: powerpc (ppc)
Kernel: Linux 2.6.9
Locale: LANG=de_DE, LC_CTYPE=de_DE (charmap=ISO-8859-1)

Versions of packages libpam-mount depends on:
ii  libc6                       2.3.2.ds1-20 GNU C Library: Shared libraries an
ii  libglib2.0-0                2.6.3-1      The GLib library of C routines
ii  libssl0.9.7                 0.9.7e-3     SSL shared libraries
ii  mount                       2.12p-3      Tools for mounting and manipulatin
ii  zlib1g                      1:1.2.2-4    compression library - runtime

-- no debconf information

--- mount.crypt 2005-03-07 17:34:09.000000000 +0100
+++ /tmp/mount.crypt    2005-03-24 12:06:42.790952752 +0100
@@ -27,7 +27,7 @@
 
 OPTIONS=""
 
 USAGE="dev dir [-o options]
 
   -o options  set the mount options  [ $OPTIONS ]"
 
@@ -93,16 +93,16 @@
        case $KEY in
                # FIXME: use cipher instead of encryption to avoid conflicting
                # with mount's built-in (cryptoloop) encryption argument.
-               cipher )
+               " cipher"|cipher )
                        CIPHER="$VAL"
                        ;;
-               keysize )
+               " keysize"|keysize )
                        KEYSIZE="$VAL"
                        ;;
-               hash )
+               " hash"|hash )
                        HASH="$VAL"
                        ;;
-               loop )
+               " loop"|loop )
                        LOOP=true
                        ;;
                * )

signature.asc
Description: Digital signature