aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorBrian Norris <computersforpeace@gmail.com>2012-02-08 13:26:20 -0800
committerArtem Bityutskiy <artem.bityutskiy@linux.intel.com>2012-02-29 09:19:17 +0200
commitc3e2250260f9ab7846cea5ae974ddb7379a8fef6 (patch)
treed927dab42854f185a97e3c5c86e3d69b2d439346
parent71c76e74661492b4f68f670514866cfc85f47089 (diff)
libmtd: perform device checking first
If we don't check for the MTD before calling `legacy_get_dev_info1', we may get errors like: libmtd: MTD subsystem is old and does not support sysfs, so MTD character device nodes have to exist libmtd: error!: "/dev/mtd2" is not a character device mtdinfo: error!: libmtd failed get MTD device 2 information error 22 (Invalid argument) So reverse the order of these two checks. Signed-off-by: Brian Norris <computersforpeace@gmail.com> Signed-off-by: Artem Bityutskiy <artem.bityutskiy@linux.intel.com>
-rw-r--r--lib/libmtd.c7
1 files changed, 3 insertions, 4 deletions
diff --git a/lib/libmtd.c b/lib/libmtd.c
index 501f3f1..c4836df 100644
--- a/lib/libmtd.c
+++ b/lib/libmtd.c
@@ -735,12 +735,11 @@ int mtd_get_dev_info1(libmtd_t desc, int mtd_num, struct mtd_dev_info *mtd)
memset(mtd, 0, sizeof(struct mtd_dev_info));
mtd->mtd_num = mtd_num;
- if (!lib->sysfs_supported)
- return legacy_get_dev_info1(mtd_num, mtd);
- else if (!mtd_dev_present(desc, mtd_num)) {
+ if (!mtd_dev_present(desc, mtd_num)) {
errno = ENODEV;
return -1;
- }
+ } else if (!lib->sysfs_supported)
+ return legacy_get_dev_info1(mtd_num, mtd);
if (dev_get_major(lib, mtd_num, &mtd->major, &mtd->minor))
return -1;