emo/procps
1
0
Fork 0
Code Issues Pull Requests Packages Releases Activity

top: address each of the most recent coverity warnings

Browse Source 
This patch attempts to supress the following warnings:

. MISSING_BREAK, TAINTED_SCALAR plus SIZEOF_MISMATCH .

Signed-off-by: Jim Warner <james.warner@comcast.net>
This commit is contained in:
Jim Warner
2020-05-27 00:00:00 -05:00
committed by Craig Small
parent ae4ad637af
commit e5ddd7ff55
1 changed files with 8 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
top/top.c
View File

@@ -3886,6 +3886,8 @@ static int config_osel (FILE *fp, char *buf, size_t size) {
if (2 != sscanf(buf, Osel_window_fmts, &wno, &tot))
goto end_oops;
if (wno < 0 || wno >= GROUPSMAX) goto end_oops;
if (tot < 0) goto end_oops;
for (i = 0; i < tot; i++) {
if (!fgets(buf, size, fp)) return 1;
@@ -3975,16 +3977,21 @@ static const char *configs_file (FILE *fp, const char *name, float *delay) {
case 'a': // 3.2.8 (former procps)
if (config_cvt(w))
return p;
// fall through
case 'f': // 3.3.0 thru 3.3.3 (ng)
SETw(w, Show_JRNUMS);
// fall through
case 'g': // from 3.3.4 thru 3.3.8
scat(w->rc.fieldscur, RCF_PLUS_H);
// fall through
case 'h': // this is release 3.3.9
w->rc.graph_cpus = w->rc.graph_mems = 0;
// these next 2 are really global, but best documented here
Rc.summ_mscale = Rc.task_mscale = SK_Kb;
// fall through
case 'i': // actual RCF_VERSION_ID
scat(w->rc.fieldscur, RCF_PLUS_J);
// fall through
case 'j': // and the next version
default:
if (strlen(w->rc.fieldscur) != sizeof(DEF_FIELDS) - 1)
@@ -5008,7 +5015,7 @@ static void other_filters (int ch) {
i = 0;
osel = w->osel_1st;
pp = alloc_c((w->osel_tot + 1) * sizeof(char **));
pp = alloc_c((w->osel_tot + 1) * sizeof(char *));
while (osel && i < w->osel_tot) {
pp[i++] = osel->raw;
osel = osel->nxt;