sergiotarxz/nuttx-apps
sergiotarxz/nuttx-apps
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

More NXFFS bugfixes

Browse Source 
git-svn-id: svn://svn.code.sf.net/p/nuttx/code/trunk@3550 42af7a65-404d-4744-a932-0658087f49c3
This commit is contained in:
patacongo 2011-05-02 01:30:30 +00:00
parent 678f9eed28
commit a14dbb3e19
1 changed files with 23 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

29
examples/nxffs/nxffs_main.c
View File

@ -318,31 +318,41 @@ static ssize_t nxffs_rdblock(int fd, FAR struct nxffs_filedesc_s *file,
size_t offset, size_t len) size_t offset, size_t len)
{ {
size_t maxio = (rand() % CONFIG_EXAMPLES_NXFFS_MAXIO) + 1; size_t maxio = (rand() % CONFIG_EXAMPLES_NXFFS_MAXIO) + 1;
size_t nbytestoread = len - offset;
ssize_t nbytesread; ssize_t nbytesread;
if (nbytestoread > maxio) if (len > maxio)
{ {
nbytestoread = maxio; len = maxio;
} }
nbytesread = read(fd, &g_fileimage[offset], nbytestoread); nbytesread = read(fd, &g_fileimage[offset], len);
if (nbytesread < 0) if (nbytesread < 0)
{ {
fprintf(stderr, "Failed to read file: %d\n", errno); fprintf(stderr, "Failed to read file: %d\n", errno);
fprintf(stderr, " File name: %s\n", file->name); fprintf(stderr, " File name: %s\n", file->name);
fprintf(stderr, " File size: %d\n", file->len); fprintf(stderr, " File size: %d\n", file->len);
fprintf(stderr, " Read offset: %d\n", offset); fprintf(stderr, " Read offset: %d\n", offset);
fprintf(stderr, " Read size: %d\n", nbytestoread); fprintf(stderr, " Read size: %d\n", len);
return ERROR; return ERROR;
} }
else if (nbytesread != nbytestoread) else if (nbytesread == 0)
{
#if 0 /* No... we do this on purpose sometimes */
fprintf(stderr, "Unexpected end-of-file:\n");
fprintf(stderr, " File name: %s\n", file->name);
fprintf(stderr, " File size: %d\n", file->len);
fprintf(stderr, " Read offset: %d\n", offset);
fprintf(stderr, " Read size: %d\n", len);
#endif
return ERROR;
}
else if (nbytesread != len)
{ {
fprintf(stderr, "Partial read:\n"); fprintf(stderr, "Partial read:\n");
fprintf(stderr, " File name: %s\n", file->name); fprintf(stderr, " File name: %s\n", file->name);
fprintf(stderr, " File size: %d\n", file->len); fprintf(stderr, " File size: %d\n", file->len);
fprintf(stderr, " Read offset: %d\n", offset); fprintf(stderr, " Read offset: %d\n", offset);
fprintf(stderr, " Read size: %d\n", nbytestoread); fprintf(stderr, " Read size: %d\n", len);
fprintf(stderr, " Bytes read: %d\n", nbytesread); fprintf(stderr, " Bytes read: %d\n", nbytesread);
} }
return nbytesread; return nbytesread;
@ -399,11 +409,12 @@ static inline int nxffs_rdfile(FAR struct nxffs_filedesc_s *file)
/* Try reading past the end of the file */ /* Try reading past the end of the file */
nbytesread = nxffs_rdblock(fd, file, ntotalread, 1024) ; nbytesread = nxffs_rdblock(fd, file, ntotalread, 1024) ;
if (nbytesread >= 0) if (nbytesread > 0)
{ {
fprintf(stderr, "Read past the end of file\n"); fprintf(stderr, "Read past the end of file\n");
fprintf(stderr, " File name: %s\n", file->name); fprintf(stderr, " File name: %s\n", file->name);
fprintf(stderr, " File size: %d\n", file->len); fprintf(stderr, " File size: %d\n", file->len);
fprintf(stderr, " Bytes read: %d\n", nbytesread);
close(fd); close(fd);
return ERROR; return ERROR;
} }
@ -461,7 +472,7 @@ static int nxffs_verifyfs(void)
} }
else else
{ {
fprintf(stderr, "File %d file OK\n", i); fprintf(stderr, "File %d: OK\n", i);
} }
} }
} }