From c1b4cf105dacee1452c725d5517c13fd41f712cf Mon Sep 17 00:00:00 2001 From: Jakub Wilk Date: Thu, 7 Mar 2019 21:42:09 +0100 Subject: setterm: disallow "default" for --ulcolor/--hbcolor Passing "default" to --ulcolor or --hbcolor worked, but it set the color to bright red. This was not a documented syntax, so let's forbid it. Signed-off-by: Jakub Wilk --- term-utils/setterm.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'term-utils') diff --git a/term-utils/setterm.c b/term-utils/setterm.c index bb13eb3c7..a8ccc3864 100644 --- a/term-utils/setterm.c +++ b/term-utils/setterm.c @@ -231,7 +231,7 @@ static int parse_ulhb_color(char **av, int *oi) color = parse_color(color_name); if (color < 0) color = strtos32_or_err(color_name, _("argument error")); - if (!is_valid_color(color)) + if (!is_valid_color(color) || color == DEFAULT) errx(EXIT_FAILURE, "%s: %s", _("argument error"), color_name); if (bright && (color == BLACK || color == GREY)) errx(EXIT_FAILURE, _("argument error: bright %s is not supported"), color_name); -- cgit v1.2.3-55-g7522