]> granicus.if.org Git - procps-ng/commitdiff
0103-top: Protect macro parameters.
authorQualys Security Advisory <qsa@qualys.com>
Thu, 1 Jan 1970 00:00:00 +0000 (00:00 +0000)
committerCraig Small <csmall@enc.com.au>
Sat, 9 Jun 2018 11:35:19 +0000 (21:35 +1000)
---------------------------- adapted for newlib branch
. the 'isBUSY' macro is quite different under newlib

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

index 5939ea889fda2be97f4c722ecb20e3473d0ab020..9bf8e54e0ca9d71ed96ba0209971e8c84776bc3c 100644 (file)
--- a/top/top.c
+++ b/top/top.c
@@ -5404,8 +5404,8 @@ static int window_show (WIN_t *q, int wmax) {
  /* the isBUSY macro determines if a task is 'active' --
     it returns true if some cpu was used since the last sample.
     ( actual 'running' tasks will be a subset of those selected ) */
- #define isBUSY(x)   (0 < PID_VAL(EU_CPU, s_int, x))
- #define winMIN(a,b) ((a < b) ? a : b)
+ #define isBUSY(x)   (0 < PID_VAL(EU_CPU, s_int, (x)))
+ #define winMIN(a,b) (((a) < (b)) ? (a) : (b))
    int i, lwin;
 
    // Display Column Headings -- and distract 'em while we sort (maybe)