diff options
author | Zephaniah E. Loss-Cutler-Hull <zephaniah@gmail.com> | 2019-02-09 05:25:48 -0800 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2019-09-21 07:15:42 +0200 |
commit | a07dd473de2757b9326b8b55ed3afc12d5a77c94 (patch) | |
tree | 066ed4be27bb48c919ce808620509993d7f29562 /scripts/Makefile | |
parent | b2c9cc27fbfa9ee4ceaf6bb1637310b9026f40d1 (diff) |
tools/power x86_energy_perf_policy: Fix argument parsing
[ Upstream commit 03531482402a2bc4ab93cf6dde46833775e035e9 ]
The -w argument in x86_energy_perf_policy currently triggers an
unconditional segfault.
This is because the argument string reads: "+a:c:dD:E:e:f:m:M:rt:u:vw" and
yet the argument handler expects an argument.
When parse_optarg_string is called with a null argument, we then proceed to
crash in strncmp, not horribly friendly.
The man page describes -w as taking an argument, the long form
(--hwp-window) is correctly marked as taking a required argument, and the
code expects it.
As such, this patch simply marks the short form (-w) as requiring an
argument.
Signed-off-by: Zephaniah E. Loss-Cutler-Hull <zephaniah@gmail.com>
Signed-off-by: Len Brown <len.brown@intel.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>
Diffstat (limited to 'scripts/Makefile')
0 files changed, 0 insertions, 0 deletions