mirror of
https://github.com/torvalds/linux.git
synced 2026-05-31 18:43:33 +02:00
perf header: Use a max number of command line args
Sashiko suggests we use some reasonable max number of args to avoid overflows when reading perf.data files, do it. Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com> Signed-off-by: Namhyung Kim <namhyung@kernel.org>
This commit is contained in:
parent
c89f35def8
commit
fc32ae6df8
|
|
@ -2795,6 +2795,9 @@ process_event_desc(struct feat_fd *ff, void *data __maybe_unused)
|
|||
return 0;
|
||||
}
|
||||
|
||||
// Some reasonable arbitrary max for the number of command line arguments
|
||||
#define MAX_CMDLINE_NR 32768
|
||||
|
||||
static int process_cmdline(struct feat_fd *ff, void *data __maybe_unused)
|
||||
{
|
||||
struct perf_env *env = &ff->ph->env;
|
||||
|
|
@ -2804,6 +2807,9 @@ static int process_cmdline(struct feat_fd *ff, void *data __maybe_unused)
|
|||
if (do_read_u32(ff, &nr))
|
||||
return -1;
|
||||
|
||||
if (nr > MAX_CMDLINE_NR)
|
||||
return -1;
|
||||
|
||||
env->nr_cmdline = nr;
|
||||
|
||||
cmdline = zalloc(ff->size + nr + 1);
|
||||
|
|
|
|||
Loading…
Reference in New Issue
Block a user