From: Jeremy Kerr <jk@ozlabs.org>
Date: Thu, 17 Apr 2014 03:47:39 +0000 (+0800)
Subject: discover/udev: NULL return from udev_enumerate_get_list_entry isn't an error
X-Git-Tag: v1.0.0~184
X-Git-Url: http://git.ozlabs.org/?p=petitboot;a=commitdiff_plain;h=b6ee414890fcd582a96f4d89e648cff5dacb1cb9;ds=sidebyside

discover/udev: NULL return from udev_enumerate_get_list_entry isn't an error

A NULL return can mean the list is empty; don't return a failure from
this case.

udev_list_entry_foreach does a check for a NULL entry, so we'll do the
correct thing in the following loop.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
---

diff --git a/discover/udev.c b/discover/udev.c
index 00b6df7..6ccb8d4 100644
--- a/discover/udev.c
+++ b/discover/udev.c
@@ -321,11 +321,6 @@ static int udev_enumerate(struct udev *udev)
 
 	list = udev_enumerate_get_list_entry(enumerate);
 
-	if (!list) {
-		pb_log("udev_enumerate_get_list_entry failed\n");
-		goto fail;
-	}
-
 	udev_list_entry_foreach(entry, list) {
 		const char *syspath;
 		struct udev_device *dev;