dovecot-2.2: acl: Empty negative rights list should be the same ...

[dovecot at dovecot.org]

details:   http://hg.dovecot.org/dovecot-2.2/rev/daa0b8e5cf2e
changeset: 17376:daa0b8e5cf2e
user:      Timo Sirainen <tss at iki.fi>
date:      Mon May 12 12:18:46 2014 +0300
description:
acl: Empty negative rights list should be the same as NULL negative rights list.
There may be other places where this should be fixed, but this at least
fixes errors with dsync where "-user" entry without any rights was attempted
to be synced in dovecot-acl (although such entry shouldn't really have
existed in the first place).

diffstat:

 src/plugins/acl/acl-api.c |  2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diffs (12 lines):

diff -r 6bb26098a45c -r daa0b8e5cf2e src/plugins/acl/acl-api.c
--- a/src/plugins/acl/acl-api.c	Sun May 11 22:52:30 2014 +0300
+++ b/src/plugins/acl/acl-api.c	Mon May 12 12:18:46 2014 +0300
@@ -363,7 +363,7 @@
 
 	if (rights->rights != NULL)
 		str_append(str, t_strarray_join(rights->rights, " "));
-	if (rights->neg_rights != NULL) {
+	if (rights->neg_rights != NULL && rights->neg_rights[0] != NULL) {
 		if (str_len(str) > 0)
 			str_append_c(str, ' ');
 		str_append_c(str, '-');