diff options
Diffstat (limited to 'src/acl.c')
-rw-r--r-- | src/acl.c | 22 |
1 files changed, 16 insertions, 6 deletions
@@ -2169,15 +2169,25 @@ sds ACLLoadFromFile(const char *filename) { server.acl_filename, linenum); } - int j; - for (j = 0; j < merged_argc; j++) { + int syntax_error = 0; + for (int j = 0; j < merged_argc; j++) { acl_args[j] = sdstrim(acl_args[j],"\t\r\n"); if (ACLSetUser(u,acl_args[j],sdslen(acl_args[j])) != C_OK) { const char *errmsg = ACLSetUserStringError(); - errors = sdscatprintf(errors, - "%s:%d: %s. ", - server.acl_filename, linenum, errmsg); - continue; + if (errno == ENOENT) { + /* For missing commands, we print out more information since + * it shouldn't contain any sensitive information. */ + errors = sdscatprintf(errors, + "%s:%d: Error in applying operation '%s': %s. ", + server.acl_filename, linenum, acl_args[j], errmsg); + } else if (syntax_error == 0) { + /* For all other errors, only print out the first error encountered + * since it might affect future operations. */ + errors = sdscatprintf(errors, + "%s:%d: %s. ", + server.acl_filename, linenum, errmsg); + syntax_error = 1; + } } } |