Bug#556388: Problem with cyrus-common-2.2.postinst script

[Bas van Sisseren]

Hello,

It would help a lot to change these lines in /usr/sbin/cyrus-makedirs:

(line 92)
find "$confdir" -print0 \( -not -user cyrus -or  -not -group mail \) | xargs
-r -0 chown cyrus:mail

(line 105)
find "$i" -print0 \( -not -user cyrus -or  -not -group mail \) | xargs -r -0
chown cyrus:mail

(line 118)
find "$sievedir" -print0 \( -not -user cyrus -or  -not -group mail \) |
xargs -r -0 chown cyrus:mail


into:

(line 92)
find "$confdir" \( -not -user cyrus -or  -not -group mail \) -print0 | xargs
-r -0 chown cyrus:mail

(line 105)
find "$i" \( -not -user cyrus -or  -not -group mail \) -print0 | xargs -r -0
chown cyrus:mail

(line 118)
find "$sievedir" \( -not -user cyrus -or  -not -group mail \) -print0 |
xargs -r -0 chown cyrus:mail


This way the file is only chowned when the user/group is incorrect. With the
-print0 statement in the front, just all files are printed, user cyrus or not..

If another security update will be released for oldstable, I would really
like this fix. Without the fix, the upgrade took me more than 1.5 hours
downtime. With the fix, this will probably be only 5 minutes.


Regards,

Bas van Sisseren

-- 
Bas van Sisseren <bas at quarantainenet.nl>
Quarantainenet