dir: Don't leak a buffer on a failed Enumerate

If, for some reasons, Enumerate encounters a hash object with a NULL
buffer pointer, that's no reason to leak the hash object. Make sure
that we DRelease it before failing

Change-Id: I9ba6c69ca524937a6068d33a0ec0a90eef5b88ea
Reviewed-on: http://gerrit.openafs.org/5659
Reviewed-by: Derrick Brashear <shadow@dementix.org>
Tested-by: BuildBot <buildbot@rampaginggeek.com>

diff --git a/src/dir/dir.c b/src/dir/dir.c
index 521145b1951e8a8628f48b3a928fedca348c105c..4e7a84401c3288aa5e284fa68b1ade9ecc4a19f1 100644 (file)
--- a/src/dir/dir.c
+++ b/src/dir/dir.c
@@ -433,8 +433,10 @@ afs_dir_EnumerateDir(dir_file_t dir, int (*proc) (void *, char *name,
                goto out;
 
            ep = (struct DirEntry *)entrybuf.data;
-           if (!ep)
+           if (!ep) {
+               DRelease(&entrybuf, 0);
                break;
+           }
 
            num = ntohs(ep->next);
            code = (*proc) (hook, ep->name, ntohl(ep->fid.vnode),