]> git.decadent.org.uk Git - nfs-utils.git/commitdiff
mountd: Plug v4root memory leak
authorChuck Lever <chuck.lever@oracle.com>
Thu, 5 Jan 2012 21:01:22 +0000 (16:01 -0500)
committerSteve Dickson <steved@redhat.com>
Thu, 5 Jan 2012 21:27:05 +0000 (16:27 -0500)
Valgrind reports that the memory allocated for eep's e_hostname field
was not being freed.  eep is not visible outside of v4root_create(),
so we don't need to strdup() that string.

Introduced by commit 3b777b0 "exports: NFSv4 pseudoroot support
routines" (Dec 1, 2009).

Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
Signed-off-by: Steve Dickson <steved@redhat.com>
utils/mountd/v4root.c

index c33a5a9eb79d690580e81825de197d799d214ded..81f813bde067777d4b357c86a2ea30a7d1bf0eee 100644 (file)
@@ -83,7 +83,7 @@ v4root_create(char *path, nfs_export *export)
        struct exportent *curexp = &export->m_export;
 
        dupexportent(&eep, &pseudo_root.m_export);
-       eep.e_hostname = strdup(curexp->e_hostname);
+       eep.e_hostname = curexp->e_hostname;
        strncpy(eep.e_path, path, sizeof(eep.e_path));
        if (strcmp(path, "/") != 0)
                eep.e_flags &= ~NFSEXP_FSID;