emo/busybox
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

ps: fix -o pid=PID,args interpreting entire "PID,args" as header

Browse Source 
procps-ng 3.3.15 does not do this.
(It could, allowing commas in headers and requiring
"ps -opid=PID -oargs" form for this case, but it does not).

function                                             old     new   delta
parse_o                                              167     190     +23

Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
This commit is contained in:
Denys Vlasenko 2021-10-07 21:55:16 +02:00
parent 421c8767ba
commit eb048a450c
1 changed files with 18 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

32
procps/ps.c
View File

@ -443,17 +443,19 @@ static void parse_o(char* opt)
opt = comma + 1;
continue;
}
break;
}
// opt points to last spec in comma separated list.
// This one can have =HEADER part.
new = new_out_t();
if (equal)
*equal = '\0';
*new = *find_out_spec(opt);
if (equal) {
*equal = '=';
new->header = equal + 1;
// opt points to last spec in comma separated list.
// This one can have =HEADER part.
new = new_out_t();
if (equal)
*equal = '\0';
*new = *find_out_spec(opt);
if (!equal)
break;
*equal++ = '=';
new->header = equal;
comma = strchr(equal, ',');
if (comma)
*comma = '\0';
// POSIX: the field widths shall be ... at least as wide as
// the header text (default or overridden value).
// If the header text is null, such as -o user=,
@ -461,10 +463,12 @@ static void parse_o(char* opt)
// default header text
if (new->header[0]) {
new->width = strlen(new->header);
print_header = 1;
}
} else
print_header = 1;
if (!comma)
break;
//*comma = ','; /* no, new->header should stay NUL-terminated */
opt = comma + 1;
}
}
static void alloc_line_buffer(void)