diff options
| author | John Crispin <john@openwrt.org> | 2015-03-05 20:25:02 +0000 |
|---|---|---|
| committer | John Crispin <john@openwrt.org> | 2015-03-05 20:25:02 +0000 |
| commit | b48b7a7689b15754938c77449d59c16fc3765b1f (patch) | |
| tree | 65f8a1e1567d8d2c74a2e33df91bac12fc1d98c4 | |
| parent | f1625b116f3fd5ff597af88e1719e6492a938d97 (diff) | |
| download | upstream-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.c | 2 |
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; } |