localectl: fix memleak, jump to finish before returning

diff --git a/src/locale/localectl.c b/src/locale/localectl.c
index c05eba0d3edebfcdedbf1cc20d47230547ddae3b..84feb25d521d0a6d612436b34a12dfc5f30f1e3b 100644 (file)
--- a/src/locale/localectl.c
+++ b/src/locale/localectl.c
@@ -354,7 +354,8 @@ static int list_locales(DBusConnection *bus, char **args, unsigned n) {
             h->locrectab_offset + h->locrectab_size > st.st_size ||
             h->sumhash_offset + h->sumhash_size > st.st_size) {
                 log_error("Invalid archive file.");
             h->locrectab_offset + h->locrectab_size > st.st_size ||
             h->sumhash_offset + h->sumhash_size > st.st_size) {
                 log_error("Invalid archive file.");

-                return -EBADMSG;
+                r = -EBADMSG;
+                goto finish;

         }
 
         e = (const struct namehashent*) ((const uint8_t*) p + h->namehash_offset);
         }
 
         e = (const struct namehashent*) ((const uint8_t*) p + h->namehash_offset);