Re: [EDT][prelink-cross] possible memory leak in opd_size

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

I will hopefully get to reviewing this and the other 3 patches by early next week.

Thanks for the submission, I'll let you know what the outcome is.

--Mark

On 5/6/15 1:33 AM, Maninder Singh wrote:
> EP-F6AA0618C49C4AEDA73BFF1B39950BAB
> 
> Hi,
> 
> we found one issue through code-walk through
> and it seems we are creating unusable OPD table , Not sure about fix, please check below fix. 
> 
> Signed-off-by: Vaneet Narang <v.narang@samsung.com>
> Signed-off-by: Maninder Singh <maninder1.s@samsung.com>
> Reviewed-by: Ajeet Yadav <ajeet.y@samsung.com>
> ---
>  trunk/src/fptr.c |    1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)
> 
> diff --git a/trunk/src/fptr.c b/trunk/src/fptr.c
> index cfe3aed..39271d5 100644
> --- a/trunk/src/fptr.c
> +++ b/trunk/src/fptr.c
> @@ -458,6 +458,7 @@ opd_size (struct prelink_info *info, GElf_Word entsize)
>     e->val = f->val;
>     e->gp = f->gp;
>     e->opd = ret | OPD_ENT_NEW;
> +   f->ent = e;
>     ret += entsize;
>        }
> 
> --
> 1.7.1
> 
> 
> Thanks and Regards,
> Maninder Singh
>