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 <maninder...@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; > -- _______________________________________________ yocto mailing list yocto@yoctoproject.org https://lists.yoctoproject.org/listinfo/yocto
