diff options
author | Felix Fietkau <nbd@openwrt.org> | 2009-09-21 19:13:34 +0000 |
---|---|---|
committer | Felix Fietkau <nbd@openwrt.org> | 2009-09-21 19:13:34 +0000 |
commit | 5b027739405dc0f0dcee3588a17df19cae11eda9 (patch) | |
tree | 093bcee8a4012917f9e2b38ae61199bcc46dc91c /package/mtd/src/mtd.c | |
parent | f2c1e05f37e6cc0e5365830fd6b2ca59ebc95d76 (diff) | |
download | upstream-5b027739405dc0f0dcee3588a17df19cae11eda9.tar.gz upstream-5b027739405dc0f0dcee3588a17df19cae11eda9.tar.bz2 upstream-5b027739405dc0f0dcee3588a17df19cae11eda9.zip |
mtd: fix wrong open() return code checks
SVN-Revision: 17666
Diffstat (limited to 'package/mtd/src/mtd.c')
-rw-r--r-- | package/mtd/src/mtd.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/package/mtd/src/mtd.c b/package/mtd/src/mtd.c index bd966ab7dc..2639b4ebb5 100644 --- a/package/mtd/src/mtd.c +++ b/package/mtd/src/mtd.c @@ -98,13 +98,13 @@ int mtd_check_open(const char *mtd) fd = mtd_open(mtd, false); if(fd < 0) { fprintf(stderr, "Could not open mtd device: %s\n", mtd); - return 0; + return -1; } if(ioctl(fd, MEMGETINFO, &mtdInfo)) { fprintf(stderr, "Could not get MTD device info from %s\n", mtd); close(fd); - return 0; + return -1; } mtdsize = mtdInfo.size; erasesize = mtdInfo.erasesize; @@ -162,7 +162,7 @@ static int mtd_check(const char *mtd) } fd = mtd_check_open(mtd); - if (!fd) + if (fd < 0) return 0; if (!buf) @@ -199,7 +199,7 @@ mtd_unlock(const char *mtd) } fd = mtd_check_open(mtd); - if(fd <= 0) { + if(fd < 0) { fprintf(stderr, "Could not open mtd device: %s\n", mtd); exit(1); } @@ -230,7 +230,7 @@ mtd_erase(const char *mtd) fprintf(stderr, "Erasing %s ...\n", mtd); fd = mtd_check_open(mtd); - if(fd <= 0) { + if(fd < 0) { fprintf(stderr, "Could not open mtd device: %s\n", mtd); exit(1); } @@ -260,7 +260,7 @@ mtd_refresh(const char *mtd) fprintf(stderr, "Refreshing mtd partition %s ... ", mtd); fd = mtd_check_open(mtd); - if(fd <= 0) { + if(fd < 0) { fprintf(stderr, "Could not open mtd device: %s\n", mtd); exit(1); } |