[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