Patchwork Fix argument parsing in x86_energy_perf_policy.

mail settings
Submitter Zephaniah E. Loss-Cutler-Hull
Date Feb. 9, 2019, 1:25 p.m.
Message ID <>
Download mbox | patch
Permalink /patch/722221/
State New
Headers show


Zephaniah E. Loss-Cutler-Hull - Feb. 9, 2019, 1:25 p.m.
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

Signed-off-by: Zephaniah E. Loss-Cutler-Hull <>



diff --git a/tools/power/x86/x86_energy_perf_policy/x86_energy_perf_policy.c b/tools/power/x86/x86_energy_perf_policy/x86_energy_perf_policy.c
index 65bbe627a425..3d442361d29c 100644
--- a/tools/power/x86/x86_energy_perf_policy/x86_energy_perf_policy.c
+++ b/tools/power/x86/x86_energy_perf_policy/x86_energy_perf_policy.c
@@ -546,7 +546,7 @@  void cmdline(int argc, char **argv)
        progname = argv[0];
-       while ((opt = getopt_long_only(argc, argv, "+a:c:dD:E:e:f:m:M:rt:u:vw",
+       while ((opt = getopt_long_only(argc, argv, "+a:c:dD:E:e:f:m:M:rt:u:vw:",
                                long_options, &option_index)) != -1) {
                switch (opt) {
                case 'a':