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 <jwilk@jwilk.net>

diff --git a/term-utils/setterm.c b/term-utils/setterm.c
index bb13eb3c74ca7768a565c44d5d34eb7c9189585f..a8ccc386482bd469d453aa076acc8d696c51e496 100644 (file)
--- 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);