aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJohn Crispin <john@openwrt.org>2015-03-05 20:25:02 +0000
committerJohn Crispin <john@openwrt.org>2015-03-05 20:25:02 +0000
commitb48b7a7689b15754938c77449d59c16fc3765b1f (patch)
tree65f8a1e1567d8d2c74a2e33df91bac12fc1d98c4
parentf1625b116f3fd5ff597af88e1719e6492a938d97 (diff)
downloadupstream-b48b7a7689b15754938c77449d59c16fc3765b1f.tar.gz
upstream-b48b7a7689b15754938c77449d59c16fc3765b1f.tar.bz2
upstream-b48b7a7689b15754938c77449d59c16fc3765b1f.zip
mtd: fix md5sum error checking
In mtd_verify(), the return value of md5sum() has been interpreted as error if nonzero, while the function returns number of processed bytes, which caused mtd_verify() to always fail. This patch fixes error checking to interpret only negative values as errors. Signed-off-by: Zefir Kurtisi <zefir.kurtisi@neratec.com> SVN-Revision: 44605
-rw-r--r--package/system/mtd/src/mtd.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/package/system/mtd/src/mtd.c b/package/system/mtd/src/mtd.c
index 72b9ddfe21..741b57b480 100644
--- a/package/system/mtd/src/mtd.c
+++ b/package/system/mtd/src/mtd.c
@@ -337,7 +337,7 @@ mtd_verify(const char *mtd, char *file)
if (quiet < 2)
fprintf(stderr, "Verifying %s against %s ...\n", mtd, file);
- if (stat(file, &s) || md5sum(file, f_md5)) {
+ if (stat(file, &s) || md5sum(file, f_md5) < 0) {
fprintf(stderr, "Failed to hash %s\n", file);
return -1;
}