From: Daniel Mack Date: Tue, 28 Jul 2015 11:58:40 +0000 (+0200) Subject: udev: check more builtins[] pointers before dereferencing X-Git-Tag: v223~10 X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=f89d10ae4;p=thirdparty%2Fsystemd.git udev: check more builtins[] pointers before dereferencing Fix some more locations where pointers from builtins[] are dereferenced before checking. Related to 8cacf69b1. --- diff --git a/src/udev/udev-builtin.c b/src/udev/udev-builtin.c index 88d1acf1685..3ebefa84e27 100644 --- a/src/udev/udev-builtin.c +++ b/src/udev/udev-builtin.c @@ -89,10 +89,16 @@ void udev_builtin_list(struct udev *udev) { } const char *udev_builtin_name(enum udev_builtin_cmd cmd) { + if (!builtins[cmd]) + return NULL; + return builtins[cmd]->name; } bool udev_builtin_run_once(enum udev_builtin_cmd cmd) { + if (!builtins[cmd]) + return -EOPNOTSUPP; + return builtins[cmd]->run_once; } @@ -116,6 +122,9 @@ int udev_builtin_run(struct udev_device *dev, enum udev_builtin_cmd cmd, const c int argc; char *argv[128]; + if (!builtins[cmd]) + return -EOPNOTSUPP; + /* we need '0' here to reset the internal state */ optind = 0; strscpy(arg, sizeof(arg), command);