[request-sponsor] Requesting sponsor for 4997138 - pn_free() should free pn_bufsize bytes

[Dan Mick]

Just to be clear, I'm working on this one.

Jeremy Teo wrote:
 Hello,
 
 as mentioned in the subject above. Trivial patch to fix this is attached.
 
 Regards, 
 Jeremy
 
 Suggested Fix:
 
 --- usr/src/uts/common/fs/pathname.c  Tue Jun 14 15:45:20 2005
 +++ usr/src/uts/common/fs/pathname.c  Wed Jun 29 14:47:11 2005
 @@ -86,7 +86,7 @@
  void
  pn_free(struct pathname *pnp)
  {
 - kmem_free(pnp-pn_buf, MAXPATHLEN);
 + kmem_free(pnp-pn_buf, pnp-pn_bufsize);
   pnp-pn_path = pnp-pn_buf = NULL;
   pnp-pn_pathlen = pnp-pn_bufsize = 0;
  }
 This message posted from opensolaris.org
 ___
 request-sponsor mailing list
 request-sponsor at opensolaris.org

[request-sponsor] Requesting sponsor for 4997138 - pn_free() should free pn_bufsize bytes

[Eric Schrock]

Jeremy -

The project that required this functionality has since switched methods
so its no longer necessary.  That being said, it seems like a reasonable
change to allow for this behavior in the future, and a simple first
bugfix.  I can sponsor this fix, but I'll be on vacation for the next
week (as will most of Sun's US employees).

- Eric

On Wed, Jun 29, 2005 at 12:06:35AM -0700, Jeremy Teo wrote:
 Hello,
 
 as mentioned in the subject above. Trivial patch to fix this is attached.
 
 Regards, 
 Jeremy
 
 Suggested Fix:
 
 --- usr/src/uts/common/fs/pathname.c  Tue Jun 14 15:45:20 2005
 +++ usr/src/uts/common/fs/pathname.c  Wed Jun 29 14:47:11 2005
 @@ -86,7 +86,7 @@
  void
  pn_free(struct pathname *pnp)
  {
 - kmem_free(pnp-pn_buf, MAXPATHLEN);
 + kmem_free(pnp-pn_buf, pnp-pn_bufsize);
   pnp-pn_path = pnp-pn_buf = NULL;
   pnp-pn_pathlen = pnp-pn_bufsize = 0;
  }
 This message posted from opensolaris.org
 ___
 request-sponsor mailing list
 request-sponsor at opensolaris.org

--
Eric Schrock, Solaris Kernel Development.

[request-sponsor] Requesting sponsor for 4997138 - pn_free() should free pn_bufsize bytes

[Jeremy Teo]

Hello,

as mentioned in the subject above. Trivial patch to fix this is attached.

Regards, 
Jeremy

Suggested Fix:

--- usr/src/uts/common/fs/pathname.cTue Jun 14 15:45:20 2005
+++ usr/src/uts/common/fs/pathname.cWed Jun 29 14:47:11 2005
@@ -86,7 +86,7 @@
 void
 pn_free(struct pathname *pnp)
 {
-   kmem_free(pnp-pn_buf, MAXPATHLEN);
+   kmem_free(pnp-pn_buf, pnp-pn_bufsize);
pnp-pn_path = pnp-pn_buf = NULL;
pnp-pn_pathlen = pnp-pn_bufsize = 0;
 }
This message posted from opensolaris.org