diff options
author | Lennart Poettering <lennart@poettering.net> | 2013-11-19 01:14:21 +0100 |
---|---|---|
committer | Lennart Poettering <lennart@poettering.net> | 2013-11-20 19:36:14 +0100 |
commit | 994e023433e020e2b3f683d5d1f2c974db580447 (patch) | |
tree | 082efdb253d69337e78500303936f07881424078 /src/libudev/libudev-device.c | |
parent | 135168183ecb23f642b4f9fb9a50410e6735c628 (diff) |
libudev: always return valid negative error codes on API functions
Diffstat (limited to 'src/libudev/libudev-device.c')
-rw-r--r-- | src/libudev/libudev-device.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/src/libudev/libudev-device.c b/src/libudev/libudev-device.c index d7f672f187..b5b07fc5de 100644 --- a/src/libudev/libudev-device.c +++ b/src/libudev/libudev-device.c @@ -536,7 +536,7 @@ int udev_device_read_db(struct udev_device *udev_device, const char *dbfile) f = fopen(dbfile, "re"); if (f == NULL) { udev_dbg(udev_device->udev, "no db file to read %s: %m\n", dbfile); - return -1; + return -errno; } udev_device->is_initialized = true; @@ -593,7 +593,7 @@ int udev_device_read_uevent_file(struct udev_device *udev_device) strscpyl(filename, sizeof(filename), udev_device->syspath, "/uevent", NULL); f = fopen(filename, "re"); if (f == NULL) - return -1; + return -errno; udev_device->uevent_loaded = true; while (fgets(line, sizeof(line), f)) { @@ -1521,13 +1521,13 @@ static int udev_device_sysattr_list_read(struct udev_device *udev_device) int num = 0; if (udev_device == NULL) - return -1; + return -EINVAL; if (udev_device->sysattr_list_read) return 0; dir = opendir(udev_device_get_syspath(udev_device)); if (!dir) - return -1; + return -errno; for (dent = readdir(dir); dent != NULL; dent = readdir(dir)) { char path[UTIL_PATH_SIZE]; |