diff options
| author | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2014-11-01 12:06:41 -0400 |
|---|---|---|
| committer | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2014-11-01 14:39:47 -0400 |
| commit | ee7122c0ec6aa11f02e9e8d94254b353f12d2c14 (patch) | |
| tree | f3beaae878a894016b8c5af3a424fb41daad788d /src/libudev/libudev.c | |
| parent | fe756ed9ec2c0fa33c30b1d9a33d745dc1593aed (diff) | |
libudev: do not accept invalid log levels
Invalid log levels lead to a assert failure later on.
https://bugs.freedesktop.org/show_bug.cgi?id=85657
Diffstat (limited to 'src/libudev/libudev.c')
| -rw-r--r-- | src/libudev/libudev.c | 19 |
1 files changed, 16 insertions, 3 deletions
diff --git a/src/libudev/libudev.c b/src/libudev/libudev.c index 7803863b2e..8464427651 100644 --- a/src/libudev/libudev.c +++ b/src/libudev/libudev.c @@ -193,7 +193,13 @@ _public_ struct udev *udev_new(void) } if (streq(key, "udev_log")) { - udev_set_log_priority(udev, util_log_priority(val)); + int prio; + + prio = util_log_priority(val); + if (prio < 0) + udev_err(udev, "/etc/udev/udev.conf:%u: invalid logging level '%s', ignoring.\n", line_nr, val); + else + udev_set_log_priority(udev, prio); continue; } } @@ -201,8 +207,15 @@ _public_ struct udev *udev_new(void) /* environment overrides config */ env = secure_getenv("UDEV_LOG"); - if (env != NULL) - udev_set_log_priority(udev, util_log_priority(env)); + if (env != NULL) { + int prio; + + prio = util_log_priority(env); + if (prio < 0) + udev_err(udev, "$UDEV_LOG specifies invalid logging level '%s', ignoring.\n", env); + else + udev_set_log_priority(udev, prio); + } return udev; } |