Re: [PATCH] Fix /proc pathname sizes on Solaris

Rainer Orth
2018-09-19T13:28:20Z

Hi Tom,

>>>>>> "Rainer" == Rainer Orth <ro@CeBiTec.Uni-Bielefeld.DE> writes:
>
> Rainer> - sprintf (pi->pathname, "/proc/%05d/lwp/%d", pid, tid);
> Rainer> + sprintf (pi->pathname, "/proc/%d/lwp/%d", pid, tid);
>
> FWIW gdb has xsnprintf which will check the length. Or I suppose it
> wouldn't be hard to just change this to be a std::string and not worry
> about lengths.

ah, good point. I'd forgotten about that here, but it's already fixed
in a largish procfs.c cleanup patch which gets rids of tons of ARI
warnings among others. Currently retesting...