Author: manu
Date: Tue Sep 26 12:15:13 2017
New Revision: 324014
URL: https://svnweb.freebsd.org/changeset/base/324014

Log:
  mountd: Avoid memory leak by freeing dp_dirp
  
  Introduced in r324007, the data alloced by strdup was never free'ed.
  While here, remove cast to caddr_t when freeing dp.
  
  Reported by:  bde
  MFC after:    1 week
  X MFC With:   r324007

Modified:
  head/usr.sbin/mountd/mountd.c

Modified: head/usr.sbin/mountd/mountd.c
==============================================================================
--- head/usr.sbin/mountd/mountd.c       Tue Sep 26 11:16:33 2017        
(r324013)
+++ head/usr.sbin/mountd/mountd.c       Tue Sep 26 12:15:13 2017        
(r324014)
@@ -2159,7 +2159,8 @@ free_dir(struct dirlist *dp)
                free_dir(dp->dp_left);
                free_dir(dp->dp_right);
                free_host(dp->dp_hosts);
-               free((caddr_t)dp);
+               free(dp->dp_dirp);
+               free(dp);
        }
 }
 
_______________________________________________
svn-src-head@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"

Reply via email to