clean up on failure.

which doesn't matter in practice, as we don't dynamically read
users files.  But it's good practice

diff --git a/src/lib/server/users_file.c b/src/lib/server/users_file.c
index 0f0d18aa475162139f662bade1c62d669f2b21b6..f3bc940bd624530dae38faab7fe0b916c02b0951 100644 (file)
--- a/src/lib/server/users_file.c
+++ b/src/lib/server/users_file.c
@@ -402,7 +402,7 @@ check_item:
                            (tmpl_regex_compile(new_map->rhs, false) < 0)) {
                                ERROR("%s[%d]: Failed compiling regular expression /%s/ - %s",
                                      file, lineno, new_map->rhs->name, fr_strerror());
-                               return -1;
+                               goto fail_entry;
                        }
 
                        goto do_insert;