This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH] Fix /proc pathname sizes on Solaris
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...
Rainer
--
-----------------------------------------------------------------------------
Rainer Orth, Center for Biotechnology, Bielefeld University