diff options
author | Gregory Nutt <gnutt@nuttx.org> | 2014-11-25 13:46:14 -0600 |
---|---|---|
committer | Gregory Nutt <gnutt@nuttx.org> | 2014-11-25 13:46:14 -0600 |
commit | f28eff61aeb62c867249c8cb21162dbd3c60cc37 (patch) | |
tree | c1c8b6d68fb43d3ae59c3eb589e6a4c44acdcd57 /nuttx/fs/nxffs | |
parent | 3b3fcf2bfa6015d890d42b5a528f7fe7476632f7 (diff) | |
download | px4-nuttx-f28eff61aeb62c867249c8cb21162dbd3c60cc37.tar.gz px4-nuttx-f28eff61aeb62c867249c8cb21162dbd3c60cc37.tar.bz2 px4-nuttx-f28eff61aeb62c867249c8cb21162dbd3c60cc37.zip |
More fixes to problems noted by cppcheck. Some are kind of risky; some are real bugs.
Diffstat (limited to 'nuttx/fs/nxffs')
-rw-r--r-- | nuttx/fs/nxffs/nxffs_dump.c | 2 | ||||
-rw-r--r-- | nuttx/fs/nxffs/nxffs_initialize.c | 2 |
2 files changed, 3 insertions, 1 deletions
diff --git a/nuttx/fs/nxffs/nxffs_dump.c b/nuttx/fs/nxffs/nxffs_dump.c index 181aaaebe..19e4d4b5c 100644 --- a/nuttx/fs/nxffs/nxffs_dump.c +++ b/nuttx/fs/nxffs/nxffs_dump.c @@ -67,7 +67,9 @@ struct nxffs_blkinfo_s off_t nblocks; off_t block; off_t offset; +#if defined(CONFIG_DEBUG) && defined(CONFIG_DEBUG_FS) bool verbose; +#endif }; /**************************************************************************** diff --git a/nuttx/fs/nxffs/nxffs_initialize.c b/nuttx/fs/nxffs/nxffs_initialize.c index 3fb9e2b77..d7d55f077 100644 --- a/nuttx/fs/nxffs/nxffs_initialize.c +++ b/nuttx/fs/nxffs/nxffs_initialize.c @@ -410,7 +410,7 @@ int nxffs_limits(FAR struct nxffs_volume_s *volume) if (!noinodes) { - while ((ret = nxffs_nextentry(volume, offset, &entry)) == OK) + while (nxffs_nextentry(volume, offset, &entry) == OK) { /* Discard the entry and guess the next offset. */ |