dovecot - Jun 2006 - 1.0rc1 location variable bug

I think I've found a bug in Dovecot 1.0 rc1 where if you have namespaces
without the "location = " setting defined, it fails to use
default_mail_env as it should.

This seems to have been broken by :-

2006-06-16 18:13  Timo Sirainen <tss at iki.fi>

* src/master/: auth-process.c, login-process.c, mail-process.c,
  main.c, master-settings.c: Settings parser nowadays returns ""
  instead of NULL when it reads an empty value from config file
 (due to some good reason I unfortunately didn't write to commit
  message and can't remember anymore). Fixed a lot of existing
  checks which checked for empty strings with NULL instead of ""
  checks.

Looks like Timo missed (at least) one!

I'm surprised nobody else has seen this. Do you all have
"location" defined?

Patch attached.

Best Wishes,
Chris

-- 
--+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+-
Christopher Wakelin,                           c.d.wakelin at reading.ac.uk
IT Services Centre, The University of Reading,  Tel: +44 (0)118 378 8439
Whiteknights, Reading, RG6 2AF, UK              Fax: +44 (0)118 975 3094
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: location_bug.patch
URL:
<http://dovecot.org/pipermail/dovecot/attachments/20060630/2f9b649f/attachment.pl>

On Fri, 2006-06-30 at 16:16 +0100, Chris Wakelin wrote:
> I think I've found a bug in Dovecot 1.0 rc1 where if you have
namespaces
> without the "location = " setting defined, it fails to use
> default_mail_env as it should.
..
> I'm surprised nobody else has seen this. Do you all have
"location" defined?
Actually someone else from IRC had already notified me of that. :) Fixed
now in CVS.

I think most people don't have namespaces defined at all.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 191 bytes
Desc: This is a digitally signed message part
URL:
<http://dovecot.org/pipermail/dovecot/attachments/20060701/67c222b4/attachment.bin>