acl: Fix array handling when removing ACL rights

The initial array size was large enough that practically this mistake
couldn't have caused wrong behavior.

diff --git a/src/plugins/acl/acl-rights.c b/src/plugins/acl/acl-rights.c
index f04c96bba730a7d440f9b8d6dc1a29508dc292d4..654ee00aefd1602358a532914de62626a34cc3ad 100644 (file)
--- a/src/plugins/acl/acl-rights.c
+++ b/src/plugins/acl/acl-rights.c
@@ -441,8 +441,8 @@ bool acl_right_names_modify(pool_t pool,
                                array_push_back(&rights, &old_rights[i]);
                }
                new_rights = empty_str_array;
-               modify_rights = array_count(&rights) == 0 ? NULL :
-                       array_front(&rights);
+               array_append_zero(&rights);
+               modify_rights = array_front(&rights);
                acl_right_names_merge(pool, &new_rights, modify_rights, TRUE);
                break;
        case ACL_MODIFY_MODE_ADD: