[PATCH] libdwfl: linux-pid-attach.c: Report actual PID (Tgid) to dwfl_attach_state.
Jan Kratochvil
jan.kratochvil at redhat.com
Thu Jan 2 15:24:04 UTC 2014
On Wed, 25 Dec 2013 23:10:50 +0100, Mark Wielaard wrote:
> On Wed, Dec 25, 2013 at 08:23:25PM +0100, Jan Kratochvil wrote:
> > On Mon, 23 Dec 2013 14:22:24 +0100, Mark Wielaard wrote:
> > > + {
> > > + pid = atoi (&buffer[5]);
> > > + break;
> > > + }
> > > + fclose (procfile);
> > > +
> > > + if (pid == 0)
> > > + return false;
> >
> > I do not understand this conditional. If "Tgid:" was not found PID will be
> > the user-specified TID, not 0.
>
> It would be zero in the very unlikely case atoi doesn't see any digits.
If you want to check /proc files validity there should be strtol and not atoi.
"Tgid: 42foo\n" still gets parsed as 42 by the code above.
Regards,
Jan
More information about the elfutils-devel
mailing list