]> granicus.if.org Git - procps-ng/commit
top: whack insidious bug surrounding auto-sized fields
authorJim Warner <james.warner@comcast.net>
Wed, 1 Jan 2020 05:00:00 +0000 (00:00 -0500)
committerCraig Small <csmall@dropbear.xyz>
Sun, 5 Jan 2020 22:13:29 +0000 (09:13 +1100)
commit00028aa23732598aecad6f1c146f3f9751372958
tree68f3a9b8546f03a69979ec95b63e1f381253272f
parentbb96fc42956c9ed926a1b958ab715f8b4a663dec
top: whack insidious bug surrounding auto-sized fields

This commit duplicates a change to that newlib branch.
However, it should be noted that such a change was not
really necessary under this master branch since proc_t
data remains valid much longer. It is being duplicated
here as documentation only. Below is the original msg.

------------------------------------------------------
This patch will eliminate a bug which is unique to our
newlib branch. It's extremely rare and only happens if
a search ('L'/'&') is initiated during the period when
fields are currently being auto-sized (AUTOX_MODE on).

This bug surfaces as either all zero results for tasks
displayed or a segmentation fault, depending upon what
fields were activated. It is caused by the timing of a
call to the <pids> 'reset' function. When called after
a task refresh, but before do_key(), this bug appears.

So this patch just ensures that 'reset' will be called
after do_key() & before the tasks have been refreshed.
------------------------------------------------------

Signed-off-by: Jim Warner <james.warner@comcast.net>
top/top.c
top/top.h