diff options
| author | Hendrik Brueckner <brueckner@linux.vnet.ibm.com> | 2016-02-19 15:21:18 +0100 |
|---|---|---|
| committer | Hendrik Brueckner <brueckner@linux.vnet.ibm.com> | 2016-02-19 15:21:18 +0100 |
| commit | 5181ab917d6407cb57043e98955f0de1614366ea (patch) | |
| tree | 5852ce7d7d900700c3ed11da5a346b12f4129dd1 /src/udev/udev-builtin-path_id.c | |
| parent | a972e1e0b3af9dbe490c6433322c635d9e5ad875 (diff) | |
udev/path_id: correct segmentation fault due to missing NULL check
Running "udevadm test-builtin path_id /sys/devices/platform/" results
in a segmentation fault.
The problem is that udev_device_get_subsystem(dev) might return NULL
in a streq() call. Solve this problem by using streq_ptr() instead.
Diffstat (limited to 'src/udev/udev-builtin-path_id.c')
| -rw-r--r-- | src/udev/udev-builtin-path_id.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/udev/udev-builtin-path_id.c b/src/udev/udev-builtin-path_id.c index b6ed45d8ba..6e9adc6e96 100644 --- a/src/udev/udev-builtin-path_id.c +++ b/src/udev/udev-builtin-path_id.c @@ -712,7 +712,7 @@ static int builtin_path_id(struct udev_device *dev, int argc, char *argv[], bool * devices do not expose their buses and do not provide a unique * and predictable name that way. */ - if (streq(udev_device_get_subsystem(dev), "block") && !supported_transport) + if (streq_ptr(udev_device_get_subsystem(dev), "block") && !supported_transport) path = mfree(path); if (path != NULL) { |