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

top: ensure sane rcfile values for the remaining stuff

Browse Source 
This will protect some remaining rcfile variables from
a possible manual editing of top's configuration file.

[ and correct two #error related boo-boos introduced ]
[ with the system default rcfile in the commit shown ]

Reference(s):
. introduced /etc/topdefaultrc
commit 55a42ae040eaa19fd3089f56d98ccbde5a9abc3a

Signed-off-by: Jim Warner <james.warner@comcast.net>
This commit is contained in:
Jim Warner 2018-06-02 00:00:00 -05:00 committed by Craig Small
parent bd7e1fb343
commit c314f9f953
1 changed files with 14 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
top/top.c
View File

@ -3247,6 +3247,12 @@ static const char *config_file (FILE *fp, const char *name, float *delay) {
} }
if (Rc.id < 'a' || Rc.id > RCF_VERSION_ID) if (Rc.id < 'a' || Rc.id > RCF_VERSION_ID)
return p; return p;
if (Rc.mode_altscr < 0 || Rc.mode_altscr > 1)
return p;
if (Rc.mode_irixps < 0 || Rc.mode_irixps > 1)
return p;
if (tmp_whole < 0)
return p;
// you saw that, right? (fscanf stickin' it to 'i') // you saw that, right? (fscanf stickin' it to 'i')
if (i < 0 || i >= GROUPSMAX) if (i < 0 || i >= GROUPSMAX)
return p; return p;
@ -3264,8 +3270,8 @@ static const char *config_file (FILE *fp, const char *name, float *delay) {
, w->rc.winname, w->rc.fieldscur)) , w->rc.winname, w->rc.fieldscur))
return p; return p;
#if PFLAGSSIZ != 100 #if PFLAGSSIZ != 100
too bad fscanf is not as flexible with his format string as snprintf // too bad fscanf is not as flexible with his format string as snprintf
error Hey, fix the above fscanf 'PFLAGSSIZ' dependency ! #error Hey, fix the above fscanf 'PFLAGSSIZ' dependency !
#endif #endif
// ensure there's been no manual alteration of fieldscur // ensure there's been no manual alteration of fieldscur
for (n = 0 ; n < EU_MAXPFLGS; n++) { for (n = 0 ; n < EU_MAXPFLGS; n++) {
@ -3278,6 +3284,8 @@ error Hey, fix the above fscanf 'PFLAGSSIZ' dependency !
return p; return p;
if (w->rc.sortindx < 0 || w->rc.sortindx >= EU_MAXPFLGS) if (w->rc.sortindx < 0 || w->rc.sortindx >= EU_MAXPFLGS)
return p; return p;
if (w->rc.maxtasks < 0)
return p;
if (w->rc.graph_cpus < 0 || w->rc.graph_cpus > 2) if (w->rc.graph_cpus < 0 || w->rc.graph_cpus > 2)
return p; return p;
if (w->rc.graph_mems < 0 || w->rc.graph_mems > 2) if (w->rc.graph_mems < 0 || w->rc.graph_mems > 2)
@ -3287,6 +3295,10 @@ error Hey, fix the above fscanf 'PFLAGSSIZ' dependency !
, &w->rc.summclr, &w->rc.msgsclr , &w->rc.summclr, &w->rc.msgsclr
, &w->rc.headclr, &w->rc.taskclr)) , &w->rc.headclr, &w->rc.taskclr))
return p; return p;
if (w->rc.summclr < 0 || w->rc.summclr > 7) return p;
if (w->rc.msgsclr < 0 || w->rc.msgsclr > 7) return p;
if (w->rc.headclr < 0 || w->rc.headclr > 7) return p;
if (w->rc.taskclr < 0 || w->rc.taskclr > 7) return p;
switch (Rc.id) { switch (Rc.id) {
case 'a': // 3.2.8 (former procps) case 'a': // 3.2.8 (former procps)