From: Jim Warner Date: Tue, 3 Dec 2013 21:00:00 +0000 (-0600) Subject: top: do not forget the fscanf %s terminating null byte X-Git-Tag: v3.3.10~124 X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=14ef47af57f48d1e51ab91e80fd8f132fabb665a;p=procps-ng top: do not forget the fscanf %s terminating null byte Reference(s): http://www.freelists.org/post/procps/procpsng-339-defects-found Signed-off-by: Jim Warner --- diff --git a/top/top.c b/top/top.c index 4d1d9126..bb15d015 100644 --- a/top/top.c +++ b/top/top.c @@ -3445,8 +3445,8 @@ static void configs_read (void) { WIN_t *w = &Winstk[i]; p = fmtmk(N_fmt(RC_bad_entry_fmt), i+1, Rc_name); - // note: "fieldscur=%__s" on next line should equal PFLAGSSIZ ! - if (2 != fscanf(fp, "%3s\tfieldscur=%80s\n" + // note: "fieldscur=%__s" on next line should equal (PFLAGSSIZ -1) ! + if (2 != fscanf(fp, "%3s\tfieldscur=%79s\n" , w->rc.winname, w->rc.fieldscur)) goto default_or_error; #if PFLAGSSIZ > 80