Dmitry,
The main problem in libproc_impl.c is that newlib->base is left uninitialized.
The check for the name length is just a safety measure. We can live without it
but I'd prefer to avoid the risk.
Carlos Santos (casantos)
Senior *Software* Maintenance Engineer
(no, I'm not going to fix your roof)
Red Hat, Inc
----- Original Message -----
> Carlos,
>
> I'd droppend changes in libproc_impl.c as it seems to be not related to
> prelink issue.
>
> PS:
>
> (from proposed patch) libproc_impl.c :
> + if (namelen >= sizeof(newlib->name)) {
>
> sizeof(newlib->name) is PATH_MAX + NAME_MAX
>
> Are you able to reproduce the issue with too long library name? if yes -
> I'll file the separate CR.
>
> -Dmitry
>
> On 2014-04-07 17:16, Carlos Santos wrote:
> > ----- Original Message -----
> >> Hi Everybody,
> >>
> >> Please review the patch
> >>
> >> http://cr.openjdk.java.net/~dsamersoff/JDK-8038392/webrev.02/
> >
> > The fix in agent/src/os/linux/libproc_impl.c contained in my original patch
> > is missing. Was it included in a separate patch?
> >
> > Carlos Santos (casantos)
> > Senior *Software* Maintenance Engineer
> > (no, I'm not going to fix your roof)
> > Red Hat, Inc
> >
>
>
> --
> Dmitry Samersoff
> Oracle Java development team, Saint Petersburg, Russia
> * I would love to change the world, but they won't give me the sources.
>
