diff options
author | Gregory Nutt <gnutt@nuttx.org> | 2014-11-25 15:26:30 -0600 |
---|---|---|
committer | Gregory Nutt <gnutt@nuttx.org> | 2014-11-25 15:26:30 -0600 |
commit | 76d6af0fd310967d8bf08d01cfc1db00e4a0c757 (patch) | |
tree | 963a7f1a3e29449a2389cfbe0053d608e7e9d16a /apps/netutils/ftpd/ftpd.c | |
parent | 6b0412f5128e2fdf94852de586b3f90cbf476240 (diff) | |
download | px4-nuttx-76d6af0fd310967d8bf08d01cfc1db00e4a0c757.tar.gz px4-nuttx-76d6af0fd310967d8bf08d01cfc1db00e4a0c757.tar.bz2 px4-nuttx-76d6af0fd310967d8bf08d01cfc1db00e4a0c757.zip |
Fixes for more issues found by cppcheck
Diffstat (limited to 'apps/netutils/ftpd/ftpd.c')
-rw-r--r-- | apps/netutils/ftpd/ftpd.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/apps/netutils/ftpd/ftpd.c b/apps/netutils/ftpd/ftpd.c index 51e1b35b1..d3d523ae4 100644 --- a/apps/netutils/ftpd/ftpd.c +++ b/apps/netutils/ftpd/ftpd.c @@ -2400,7 +2400,7 @@ static int ftpd_list(FAR struct ftpd_session_s *session, unsigned int opton) free(abspath); } - return OK; + return ret; } /**************************************************************************** @@ -2768,7 +2768,7 @@ static int ftpd_command_eprt(FAR struct ftpd_session_s *session) left = 0; right = strlen(session->param); - if (right <= 0) + if (right < 1) { /* no message ? */ @@ -2789,7 +2789,7 @@ static int ftpd_command_eprt(FAR struct ftpd_session_s *session) left++; } - if (right <= 0 || left > right) + if (right < 1 || left > right) { /* Invalid format */ |