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 | |
| parent | 6b0412f5128e2fdf94852de586b3f90cbf476240 (diff) | |
| download | nuttx-76d6af0fd310967d8bf08d01cfc1db00e4a0c757.tar.gz nuttx-76d6af0fd310967d8bf08d01cfc1db00e4a0c757.tar.bz2 nuttx-76d6af0fd310967d8bf08d01cfc1db00e4a0c757.zip | |
Fixes for more issues found by cppcheck
Diffstat (limited to 'apps/netutils/ftpd')
| -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 */ |