Re: [prelink-cross][PATCH 1/1] rtld: remove if check before free

[Mark Hatle <mark.hatle@windriver.com>]

I did not merge this patch.  While it's not strictly necessary to get for
allocation before free.  It's also not a bad practice to do so.

--Mark

On 6/23/15 7:28 AM, Maninder Singh wrote:
> No need of explicit NULL check before free
> 
> Signed-off-by: Maninder Singh <maninder1.s@samsung.com>
> ---
>  trunk/src/rtld/rtld.c |    1 -
>  1 files changed, 0 insertions(+), 1 deletions(-)
> 
> diff --git a/trunk/src/rtld/rtld.c b/trunk/src/rtld/rtld.c
> index 35dfde7..3ed1db3 100644
> --- a/trunk/src/rtld/rtld.c
> +++ b/trunk/src/rtld/rtld.c
> @@ -1270,7 +1270,6 @@ process_one_dso (DSO *dso, int host_paths)
>  		       (uint32_t) l->l_map_start);
>  	}
>  
> -      if (filename)
>  	free (filename);
>  
>        cur_dso_ent = cur_dso_ent->next;
>