<div dir="ltr">yeah, it was actually the ownership of the line allocations that was most difficult for me to convince myself was correct here (which is why far more effort went into rewriting that than the one-line sscanf).<div><br></div><div>i'm in the "just use cat" club too, but this was another thing that upsets the static analyzer that's easy to just think about once and not worry about again...</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, Sep 26, 2021 at 1:03 PM Rob Landley <<a href="mailto:rob@landley.net">rob@landley.net</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 9/25/21 8:44 PM, enh via Toybox wrote:<br>
> Remove unbounded scanfs.<br>
<br>
*shrug* I tend to trust data that comes from the kernel via a mount point put<br>
there by root, but if it's bothering you...<br>
<br>
(I've never actually used pmaps, I always just look at /proc/$PID/maps or<br>
/proc/$PID/smaps directly...)<br>
<br>
Rob<br>
</blockquote></div>