More FTP client debug fixes

git-svn-id: svn://svn.code.sf.net/p/nuttx/code/trunk@3662 42af7a65-404d-4744-a932-0658087f49c3

1 files changed, 29 insertions, 5 deletions

diff --git a/apps/netutils/ftpc/ftpc_pwd.c b/apps/netutils/ftpc/ftpc_pwd.c
index f6bde0e13..809002c9b 100644
--- a/apps/netutils/ftpc/ftpc_pwd.c
+++ b/apps/netutils/ftpc/ftpc_pwd.c
@@ -37,7 +37,7 @@
  * Included Files
  ****************************************************************************/
 
-#include <nuttx/config.h>
+#include "ftpc_config.h"
 
 #include <stdlib.h>
 #include <string.h>
@@ -86,26 +86,50 @@ FAR char *ftpc_pwd(SESSION handle)
   FAR char *end;
   FAR char *pwd;
   FAR char *ptr;
+  int len;
   int ret;
 
+  /* Send the PWD command */
+
   ret = ftpc_cmd(session, "PWD");
+
+  /* Response is like: 257 "/home/gnutt" (from vsftpd).
+   *
+   * Extract the quoated path name into allocated memory.
+   */
+
   start = strchr(session->reply, '\"');
   if (!start)
     {
       ndbg("Opening quote not found\n");
       return NULL;
     }
-
   start++;
+ 
   end = strchr(start, '\"');
   if (!end)
     {
-      ndbg("Clsoing quote not found\n");
+      ndbg("Closing quote not found\n");
+      return NULL;
+    }
+
+  /* Allocate memory for the path name */
+ 
+  len = end - start;
+  pwd = (char *)malloc(len + 1);
+  if (!pwd)
+    {
+      ndbg("Failed to allocate string\n");
       return NULL;
     }
+ 
+  /* Copy the string into the allocated memory */
+
+  memcpy(pwd, start, len);
+  pwd[len] = '\0';
+
+  /* Remove any trailing slashes that the server may have added */
 
-  pwd = (char *)malloc(end-start+1);
-  strncpy(pwd, start, end-start);
   ftpc_stripslash(pwd);
 
   /* Change DOS style directory separator ('\') to UNIX style ('/') */