]> granicus.if.org Git - procps-ng/commitdiff
top: correct a flaw in the support for pids monitoring
authorJim Warner <james.warner@comcast.net>
Thu, 8 Oct 2015 05:00:00 +0000 (00:00 -0500)
committerCraig Small <csmall@enc.com.au>
Fri, 9 Oct 2015 10:35:28 +0000 (21:35 +1100)
Ever since top was adapted to the new <pids> interface
there has been a bug that would cause an abnormal exit
when the '-p' argument contained *no* valid pids. This
was never revealed until now since the QA folks tested
only with valid, existing pids. (bunch of morons, eh?)

And even though the program author is blameless he has
taken it upon himself to clean up after the QA jokers.

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

index e1537f0ad184ffd7888fc029d218a3cd421cfae0..253b796140372a251035f1460200316b3406b7c3 100644 (file)
--- a/top/top.c
+++ b/top/top.c
@@ -2248,7 +2248,7 @@ static void procs_refresh (void) {
  #define nALGN2(n,m) ((n + m - 1) & ~(m - 1))    // with power of 2 align
  #define n_reap  Pids_reap->counts.total         // maintained by newlib
    static double uptime_sav;
-   static int n_alloc;                           // size of windows stacks arrays
+   static int n_alloc = -1;                      // size of windows stacks arrays
    double uptime_cur;
    float et;
    int i;