main: don't segfault when --log-color is passed without parameter
This commit is contained in:
parent
c952c6ece2
commit
d0b170c813
22
src/main.c
22
src/main.c
|
@ -412,19 +412,25 @@ static int parse_argv(int argc, char *argv[]) {
|
||||||
|
|
||||||
case ARG_LOG_COLOR:
|
case ARG_LOG_COLOR:
|
||||||
|
|
||||||
if ((r = log_show_color_from_string(optarg)) < 0) {
|
if (optarg) {
|
||||||
log_error("Failed to parse log color setting %s.", optarg);
|
if ((r = log_show_color_from_string(optarg)) < 0) {
|
||||||
return r;
|
log_error("Failed to parse log color setting %s.", optarg);
|
||||||
}
|
return r;
|
||||||
|
}
|
||||||
|
} else
|
||||||
|
log_show_color(true);
|
||||||
|
|
||||||
break;
|
break;
|
||||||
|
|
||||||
case ARG_LOG_LOCATION:
|
case ARG_LOG_LOCATION:
|
||||||
|
|
||||||
if ((r = log_show_location_from_string(optarg)) < 0) {
|
if (optarg) {
|
||||||
log_error("Failed to parse log location setting %s.", optarg);
|
if ((r = log_show_location_from_string(optarg)) < 0) {
|
||||||
return r;
|
log_error("Failed to parse log location setting %s.", optarg);
|
||||||
}
|
return r;
|
||||||
|
}
|
||||||
|
} else
|
||||||
|
log_show_location(true);
|
||||||
|
|
||||||
break;
|
break;
|
||||||
|
|
||||||
|
|
Loading…
Reference in a new issue