linux-kernel - Re: [PATCH] cpupower: Fix segfault due to incorrect getopt

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CA+5PVA7k8F8tEOz1p4O1Mh9xP3xuixPrwYtPQfvA3o7mmwb_MA@mail.gmail.com>
Date:	Tue, 17 Dec 2013 10:05:53 -0500
From:	Josh Boyer <jwboyer@...oraproject.org>
To:	Dominik Brodowski <linux@...inikbrodowski.net>,
	Thomas Renninger <trenn@...e.de>
Cc:	"Linux-Kernel@...r. Kernel. Org" <linux-kernel@...r.kernel.org>,
	Andrew Morton <akpm@...ux-foundation.org>,
	Linus Torvalds <torvalds@...ux-foundation.org>
Subject: Re: [PATCH] cpupower: Fix segfault due to incorrect getopt_long arugments

On Fri, Oct 11, 2013 at 8:45 AM, Josh Boyer <jwboyer@...hat.com> wrote:
> If a user calls 'cpupower set --perf-bias 15', the process will end with a
> SIGSEGV in libc because cpupower-set passes a NULL optarg to the atoi call.
> This is because the getopt_long structure currently has all of the options
> as having an optional_argument when they really have a required argument.
> We change the structure to use required_argument to match the short options
> and it resolves the issue.
>
> This fixes https://bugzilla.redhat.com/show_bug.cgi?id=1000439
>
> Cc: stable@...r.kernel.org
> Signed-off-by: Josh Boyer <jwboyer@...oraproject.org>

Ping?  I sent this 2 months ago and never got a reply.  We're still
carrying it in Fedora.

josh

> ---
>  tools/power/cpupower/utils/cpupower-set.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/tools/power/cpupower/utils/cpupower-set.c b/tools/power/cpupower/utils/cpupower-set.c
> index dc4de37..bcf1d2f 100644
> --- a/tools/power/cpupower/utils/cpupower-set.c
> +++ b/tools/power/cpupower/utils/cpupower-set.c
> @@ -18,9 +18,9 @@
>  #include "helpers/bitmask.h"
>
>  static struct option set_opts[] = {
> -       { .name = "perf-bias",  .has_arg = optional_argument,   .flag = NULL,   .val = 'b'},
> -       { .name = "sched-mc",   .has_arg = optional_argument,   .flag = NULL,   .val = 'm'},
> -       { .name = "sched-smt",  .has_arg = optional_argument,   .flag = NULL,   .val = 's'},
> +       { .name = "perf-bias",  .has_arg = required_argument,   .flag = NULL,   .val = 'b'},
> +       { .name = "sched-mc",   .has_arg = required_argument,   .flag = NULL,   .val = 'm'},
> +       { .name = "sched-smt",  .has_arg = required_argument,   .flag = NULL,   .val = 's'},
>         { },
>  };
>
> --
> 1.8.3.1
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@...r.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/