auth: plug auth realms memory leaks

The function _afsconf_FreeRealms, called by afsconf_CloseInternal, leaks
two afsconf_realms structures.

The function _afsconf_LoadRealms also leaks those two structures when it
fails.

These memory leaks were discovered with valgrind.

Change-Id: I1436ce21609951bc3433b6c91221cc45e78881bc
Reviewed-on: https://gerrit.openafs.org/13395
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Andrew Deason <adeason@sinenomine.net>
Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>
Reviewed-by: Mark Vitale <mvitale@sinenomine.net>

diff --git a/src/auth/realms.c b/src/auth/realms.c
index db5fd31..6508047 100644 (file)
--- a/src/auth/realms.c
+++ b/src/auth/realms.c
@@ -411,11 +411,13 @@ _afsconf_FreeRealms(struct afsconf_dir *dir)
        if (dir->local_realms) {
            destroy_tree(dir->local_realms);
            free_realm_entries(&dir->local_realms->list);
+           free(dir->local_realms);
            dir->local_realms = NULL;
        }
        if (dir->exclusions) {
            destroy_tree(dir->exclusions);
            free_realm_entries(&dir->exclusions->list);
+           free(dir->exclusions);
            dir->exclusions = NULL;
        }
     }
@@ -485,10 +487,12 @@ _afsconf_LoadRealms(struct afsconf_dir *dir)
     if (local_realms) {
        destroy_tree(local_realms);
        free_realm_entries(&local_realms->list);
+       free(local_realms);
     }
     if (exclusions) {
        destroy_tree(dir->exclusions);
        free_realm_entries(&exclusions->list);
+       free(exclusions);
     }
     return code;
 }