]> www.infradead.org Git - mtd-utils.git/commitdiff
nand-utils: nanddump: Use a specific variable for the buffer size
authorMiquel Raynal <miquel.raynal@bootlin.com>
Mon, 26 Aug 2024 09:46:20 +0000 (11:46 +0200)
committerDavid Oberhollenzer <david.oberhollenzer@sigma-star.at>
Tue, 8 Oct 2024 06:14:30 +0000 (08:14 +0200)
The read buffer size happen to be as big as the bs variable, but this is
going to change. When accessing the buffer size, use a specific variable
instead.

Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
Signed-off-by: David Oberhollenzer <david.oberhollenzer@sigma-star.at>
nand-utils/nanddump.c

index 47539f528aa2d98587b6b8afde340fcfb0def2d3..346af1e1f8c0b9a8c41ce56e89f89c50688d1009 100644 (file)
@@ -332,7 +332,7 @@ static int ofd_write(int ofd, const void *buf, size_t nbyte)
  */
 int main(int argc, char * const argv[])
 {
-       long long ofs, end_addr = 0;
+       long long ofs, end_addr = 0, readbuf_sz;
        long long blockstart = 1;
        int i, fd, ofd = 0, bs, badblock = 0;
        struct mtd_dev_info mtd;
@@ -362,8 +362,9 @@ int main(int argc, char * const argv[])
                return errmsg("mtd_get_dev_info failed");
 
        /* Allocate buffers */
+       readbuf_sz = mtd.min_io_size;
        oobbuf = xmalloc(mtd.oob_size);
-       readbuf = xmalloc(mtd.min_io_size);
+       readbuf = xmalloc(readbuf_sz);
 
        if (noecc)  {
                if (ioctl(fd, MTDFILEMODE, MTD_FILE_MODE_RAW) != 0) {
@@ -462,7 +463,7 @@ int main(int argc, char * const argv[])
                                        end_addr = mtd.size;
                                continue;
                        }
-                       memset(readbuf, 0xff, bs);
+                       memset(readbuf, 0xff, readbuf_sz);
                } else {
                        /* Read page data and exit on failure */
                        if (mtd_read(&mtd, fd, ofs / mtd.eb_size, ofs % mtd.eb_size, readbuf, bs)) {