Package: postgresql
Version: 7.4.7-6sarge1
Severity: normal

46: if ($method eq "ident" or $method = "pam") {

The = should be eq.

As it stands, not only does this test always succeed (so the
test later in the script which verifies that the parameter is
only set in the ident or pam always determines that there's no
problem, even if there is one), but the $method variable, which
is used in generating the new pg_hba.conf file, has been changed,
meaning that configurations which worked correctly, and would
continue to work correctly if the authentication method was
unchanged, now cease working and the postinst itself ends up
having problems in some situations.

Further, the test for "excess" parameters fails to deal with the
optional extra parameter for the password, md5 (and perhaps crypt?)
methods. So, not only should = be replaced with eq, but
additional tests for password, md5 and, perhaps, crypt should be
added.

- Raz


-- System Information:
Debian Release: 3.1
Architecture: i386 (i686)
Kernel: Linux 2.4.27-2-686-smp
Locale: LANG=C, LC_CTYPE=C (charmap=ANSI_X3.4-1968)

Versions of packages postgresql depends on:
ii  adduser          3.63                    Add and remove users and groups
ii  debconf [debconf 1.4.30.13               Debian configuration management sy
ii  debianutils      2.8.4                   Miscellaneous utilities specific t
ii  dpkg             1.10.28                 Package maintenance system for Deb
ii  libc6            2.3.2.ds1-22            GNU C Library: Shared libraries an
ii  libcomerr2       1.37-2sarge1            common error description library
ii  libkrb53         1.3.6-2sarge2           MIT Kerberos runtime libraries
ii  libpam0g         0.76-22                 Pluggable Authentication Modules l
ii  libperl5.8       5.8.4-8                 Shared Perl library
ii  libpq3           7.4.7-6sarge1           PostgreSQL C client library
ii  libreadline4     4.3-11                  GNU readline and history libraries
ii  libssl0.9.7      0.9.7e-3                SSL shared libraries
ii  mailx            1:8.1.2-0.20040524cvs-4 A simple mail user agent
ii  postgresql-clien 7.4.7-6sarge1           front-end programs for PostgreSQL
ii  procps           1:3.2.1-2               The /proc file system utilities
ii  python2.3        2.3.5-3                 An interactive high-level object-o
ii  ucf              1.17                    Update Configuration File: preserv
ii  zlib1g           1:1.2.2-4.sarge.2       compression library - runtime

-- debconf information:
  postgresql/initdb/location: /var/lib/postgres/data
  postgresql/upgrade/preserve_location: $PGDATA/..
  postgresql/settings/day_month_order: US
  postgresql/upgrade/policy: true
  postgresql/settings/locale: C
  postgresql/enable_lang: true
  postgresql/purge_data_too: false
  postgresql/very_old_version_warning: true
  postgresql/upgrade/dump_location: $PGDATA/..
  postgresql/convert-pg_hba.conf: true


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

Reply via email to