Gedare.. are these from Coverity Scan? They seem
remarkably picky otherwise. :)

If they are, ask when you want another run submitted.

--joel

On 9/5/2013 12:10 PM, Gedare Bloom wrote:
Fix the type of "cmpltd" to be ssize_t so that assigning it to -1
will terminate processing as intended.
---
  cpukit/libfs/src/dosfs/msdos_dir.c |    2 +-
  1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/cpukit/libfs/src/dosfs/msdos_dir.c 
b/cpukit/libfs/src/dosfs/msdos_dir.c
index 253203c..d7f6823 100644
--- a/cpukit/libfs/src/dosfs/msdos_dir.c
+++ b/cpukit/libfs/src/dosfs/msdos_dir.c
@@ -78,7 +78,7 @@ msdos_dir_read(rtems_libio_t *iop, void *buffer, size_t count)
      const size_t       buf_size = converter->buffer.size;
      uint32_t           start = 0;
      ssize_t            ret = 0;
-    uint32_t           cmpltd = 0;
+    ssize_t            cmpltd = 0;
      uint32_t           j = 0, i = 0;
      uint32_t           bts2rd = 0;
      uint32_t           cur_cln = 0;


--
Joel Sherrill, Ph.D.             Director of Research & Development
joel.sherr...@oarcorp.com        On-Line Applications Research
Ask me about RTEMS: a free RTOS  Huntsville AL 35805
Support Available                (256) 722-9985

_______________________________________________
rtems-devel mailing list
rtems-devel@rtems.org
http://www.rtems.org/mailman/listinfo/rtems-devel

Reply via email to