| Message ID | CAGKNqbE+bWgfe60nh5LpjWcu8t5J66Fx7tpjw7FYrCqw_3Wu_Q@mail.gmail.com |
|---|---|
| State | Superseded |
| Headers | show |
On 12/14/2016 12:59 AM, Jon Fether wrote: > Add correct casting for filestat.st_size in flashcp.c. While the > interim status updates had correct casting from commit 08b243, the > final update was not. > > Signed-off-by: Jonathan Fether <jonf@mds.com> > --- > diff --git a/misc-utils/flashcp.c b/misc-utils/flashcp.c > index 6594a45..af3c88d 100644 > --- a/misc-utils/flashcp.c > +++ b/misc-utils/flashcp.c > @@ -386,8 +386,8 @@ int main (int argc,char *argv[]) > if (flags & FLAG_VERBOSE) > log_printf (LOG_NORMAL, > "\rVerifying data: %lluk/%lluk (100%%)\n", > - KB (filestat.st_size), > - KB (filestat.st_size)); > + KB ((unsigned long long)filestat.st_size), > + KB ((unsigned long long)filestat.st_size)); > DEBUG("Verified %d / %lluk bytes\n",written,(unsigned long > long)filestat.st_size); Good catch! The cast used everywhere else in the file is indeed missing there. Unfortunately however, the format of your patch seems to be broken. Thanks, David ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/
My apologies, I used git diff where I see I needed to use git-format-patch. I will re-post the patch from that. Best Regards, -Jon On Tue, Dec 13, 2016 at 5:16 PM, David Oberhollenzer <david.oberhollenzer@sigma-star.at> wrote: > On 12/14/2016 12:59 AM, Jon Fether wrote: >> Add correct casting for filestat.st_size in flashcp.c. While the >> interim status updates had correct casting from commit 08b243, the >> final update was not. >> >> Signed-off-by: Jonathan Fether <jonf@mds.com> >> --- >> diff --git a/misc-utils/flashcp.c b/misc-utils/flashcp.c >> index 6594a45..af3c88d 100644 >> --- a/misc-utils/flashcp.c >> +++ b/misc-utils/flashcp.c >> @@ -386,8 +386,8 @@ int main (int argc,char *argv[]) >> if (flags & FLAG_VERBOSE) >> log_printf (LOG_NORMAL, >> "\rVerifying data: %lluk/%lluk (100%%)\n", >> - KB (filestat.st_size), >> - KB (filestat.st_size)); >> + KB ((unsigned long long)filestat.st_size), >> + KB ((unsigned long long)filestat.st_size)); >> DEBUG("Verified %d / %lluk bytes\n",written,(unsigned long >> long)filestat.st_size); > > Good catch! The cast used everywhere else in the file is indeed > missing there. > > Unfortunately however, the format of your patch seems to be broken. > > > Thanks, > > David > ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/
diff --git a/misc-utils/flashcp.c b/misc-utils/flashcp.c index 6594a45..af3c88d 100644 --- a/misc-utils/flashcp.c +++ b/misc-utils/flashcp.c @@ -386,8 +386,8 @@ int main (int argc,char *argv[]) if (flags & FLAG_VERBOSE) log_printf (LOG_NORMAL, "\rVerifying data: %lluk/%lluk (100%%)\n", - KB (filestat.st_size), - KB (filestat.st_size)); + KB ((unsigned long long)filestat.st_size), + KB ((unsigned long long)filestat.st_size)); DEBUG("Verified %d / %lluk bytes\n",written,(unsigned long long)filestat.st_size);
Add correct casting for filestat.st_size in flashcp.c. While the interim status updates had correct casting from commit 08b243, the final update was not. Signed-off-by: Jonathan Fether <jonf@mds.com> --- exit (EXIT_SUCCESS); -- 2.8.1 ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/