2007-08-21 16:19:12

by Chuck Lever

[permalink] [raw]
Subject: [PATCH 1/4] mount.nfs: mount's background daemon should free "mount_point" properly

Signed-off-by: Chuck Lever <[email protected]>
---

utils/mount/mount.c | 3 ++-
1 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/utils/mount/mount.c b/utils/mount/mount.c
index d299358..40e7958 100644
--- a/utils/mount/mount.c
+++ b/utils/mount/mount.c
@@ -557,7 +557,7 @@ int main(int argc, char *argv[])

/*
* Parent exits immediately with success. Make
- * sure not to free "mount_point"
+ * sure parent doesn't free "mount_point".
*/
if (fork() > 0)
exit(0);
@@ -568,6 +568,7 @@ int main(int argc, char *argv[])
if (verbose && mnt_err)
printf(_("%s: giving up \"%s\"\n"),
progname, spec);
+ free(mount_point);
exit(0);
}



-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems? Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >> http://get.splunk.com/
_______________________________________________
NFS maillist - [email protected]
https://lists.sourceforge.net/lists/listinfo/nfs


2007-08-21 22:19:44

by Steinar H. Gunderson

[permalink] [raw]
Subject: Re: [PATCH 1/4] mount.nfs: mount's background daemon should free "mount_point" properly

On Tue, Aug 21, 2007 at 12:19:02PM -0400, Chuck Lever wrote:
> * Parent exits immediately with success. Make
> - * sure not to free "mount_point"
> + * sure parent doesn't free "mount_point".

What made this change?

/* Steinar */
--
Homepage: http://www.sesse.net/

-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems? Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >> http://get.splunk.com/
_______________________________________________
NFS maillist - [email protected]
https://lists.sourceforge.net/lists/listinfo/nfs