Re: [PATCH] libsepol: fix memory leak in sepol_bool_query()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, 2017-09-13 at 10:16 +0200, Jan.Zarsky.jzarsky@xxxxxxxxxx wrote:
> From: Jan Zarsky <jzarsky@xxxxxxxxxx>
> 
> When sepol_bool_query() returns NULL response, variable name is not
> freed. Fix this by calling free() before returning.
> 
> Signed-off-by: Jan Zarsky <jzarsky@xxxxxxxxxx>

Thanks, applied.

> ---
>  libsepol/src/booleans.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/libsepol/src/booleans.c b/libsepol/src/booleans.c
> index c914a28..30fcf29 100644
> --- a/libsepol/src/booleans.c
> +++ b/libsepol/src/booleans.c
> @@ -155,6 +155,7 @@ int sepol_bool_query(sepol_handle_t * handle,
>  	booldatum = hashtab_search(policydb->p_bools.table, name);
>  	if (!booldatum) {
>  		*response = NULL;
> +		free(name);
>  		return STATUS_SUCCESS;
>  	}
>  



[Index of Archives]     [Selinux Refpolicy]     [Linux SGX]     [Fedora Users]     [Fedora Desktop]     [Yosemite Photos]     [Yosemite Camping]     [Yosemite Campsites]     [KDE Users]     [Gnome Users]

  Powered by Linux