Re: [liblo] A memory leak in lo_url_get_path()

[Stephen S. <rad...@gm...>]

On Wed, Oct 29, 2008 at 1:06 AM, Kentaro Fukuchi <fu...@me...> wrote:
> A memory leak fix of lo_url_get_path() follows.
>
> I was going to add a new test to detect this leak (with a memory leak checker)
> and found another issue of the URI form of OSC addressing for UNIX pipe. I'll
> report it in another mail.
>
> diff --git a/src/address.c b/src/address.c
> index 44a801e..d1a5287 100644
> --- a/src/address.c
> +++ b/src/address.c
> @@ -319,6 +319,8 @@ char *lo_url_get_path(const char *url)
>     }
>
>     /* doesnt look like an OSC URL with port number and path*/
> +    free(path);
> +
>     return NULL;
>  }

Applied.  Good stuff!
I'll try to take the time to explicitly hunt down more of these kinds
of things...
Seems like you're finding a lot of dangling mallocs.

Steve