refactor internal handle list and set filesystem override priority

Author: JacobBarthelmeh

ide/IAR-EWARM/Projects/lib/myFilesystem.h

@@ -144,12 +144,12 @@ static int SFTP_GetAttributes(void* fs, const char* fileName,
 }
 
 static int SFTP_GetAttributes_Handle(void* ssh, byte* handle, int handleSz,
-        void* atr) {
-            (void)ssh; (void)handle; (void)handleSz;
+        char* name, void* atr) {
+            (void)ssh; (void)handle; (void)handleSz; (void)name;
 
         return 0;
 }
 
 #endif /*  WOLFSSH_USER_FILESYSTEM */
 
-#endif
+#endif

ide/STM32CUBE/myFilesystem.h

@@ -141,8 +141,8 @@ static inline int SFTP_GetAttributes(void* fs, const char* fileName,
 }
 
 static inline int SFTP_GetAttributes_Handle(void* ssh, byte* handle, int handleSz,
-        void* atr) {
-            (void)ssh; (void)handle; (void)handleSz;
+        char* name, void* atr) {
+            (void)ssh; (void)handle; (void)handleSz; (void)name;
 
         return 0;
 }

src/wolfsftp.c

@@ -994,10 +994,7 @@ static INLINE int SFTP_GetSz(byte* buf, word32* sz,
 
 #ifndef NO_WOLFSSH_SERVER
 
-#if defined(MICROCHIP_MPLAB_HARMONY) || defined(WOLFSSH_ZEPHYR) || \
-    defined(WOLFSSH_FATFS) || defined(FREESCALE_MQX) || \
-    defined(USE_WINDOWS_API) || defined(WOLFSSL_NUCLEUS) || \
-    !defined(WOLFSSH_USER_FILESYSTEM)
+#if !defined(WOLFSSH_USER_FILESYSTEM)
 static int SFTP_GetAttributes(void* fs, const char* fileName,
         WS_SFTP_FILEATRB* atr, byte noFollow, void* heap);
 static int SFTP_GetAttributes_Handle(WOLFSSH* ssh, byte* handle, int handleSz,
@@ -3138,7 +3135,9 @@ static int wolfSSH_SFTPNAME_readdir(WOLFSSH* ssh, WDIR* dir, WS_SFTPNAME* out,
 }
 
 #elif defined(MICROCHIP_MPLAB_HARMONY)
+#ifndef WOLFSSH_USER_FILESYSTEM
 int SFTP_GetAttributesStat(WS_SFTP_FILEATRB* atr, WSTAT_T* stats);
+#endif
 
 /* helper function that gets file information from reading directory
  *
@@ -4413,7 +4412,10 @@ int SFTP_RemoveHandleNode(WOLFSSH* ssh, byte* handle, word32 handleSz)
 #endif /* WOLFSSH_STOREHANDLE */
 
 
-#ifdef WOLFSSL_NUCLEUS
+#if defined(WOLFSSH_USER_FILESYSTEM)
+    /* User-defined I/O support */
+
+#elif defined(WOLFSSL_NUCLEUS)
 
 #ifndef NO_WOLFSSH_MKTIME
 
@@ -4536,22 +4538,15 @@ int SFTP_GetAttributes(void* fs, const char* fileName, WS_SFTP_FILEATRB* atr,
  * returns WS_SUCCESS on success
  */
 int SFTP_GetAttributes_Handle(WOLFSSH* ssh, byte* handle, int handleSz,
-        WS_SFTP_FILEATRB* atr)
+        char* name, WS_SFTP_FILEATRB* atr)
 {
     DSTAT stats;
-    WS_HANDLE_LIST* cur;
 
     if (handle == NULL || atr == NULL) {
         return WS_FATAL_ERROR;
     }
 
-    cur = SFTP_GetHandleNode(ssh, handle, handleSz);
-    if (cur == NULL) {
-        WLOG(WS_LOG_SFTP, "Unknown handle");
-        return WS_BAD_FILE_E;
-    }
-
-    if (WSTAT(ssh->fs, cur->name, &stats) != NU_SUCCESS) {
+    if (WSTAT(ssh->fs, name, &stats) != NU_SUCCESS) {
         return WS_FATAL_ERROR;
     }
 
@@ -4705,10 +4700,9 @@ int SFTP_GetAttributes(void* fs, const char* fileName, WS_SFTP_FILEATRB* atr,
  * Fills out a WS_SFTP_FILEATRB structure
  * returns WS_SUCCESS on success */
 int SFTP_GetAttributes_Handle(WOLFSSH* ssh, byte* handle, int handleSz,
-                              WS_SFTP_FILEATRB* atr)
+                              char* name, WS_SFTP_FILEATRB* atr)
 {
     int err;
-    WS_HANDLE_LIST* cur;
     MQX_FILE_PTR mfs_ptr;
     MFS_SEARCH_DATA search_data;
     MFS_SEARCH_PARAM search;
@@ -4718,14 +4712,8 @@ int SFTP_GetAttributes_Handle(WOLFSSH* ssh, byte* handle, int handleSz,
     }
     mfs_ptr = (MQX_FILE_PTR)ssh->fs;
 
-    cur = SFTP_GetHandleNode(ssh, handle, handleSz);
-    if (cur == NULL) {
-        WLOG(WS_LOG_SFTP, "Unknown handle");
-        return WS_BAD_FILE_E;
-    }
-
     search.ATTRIBUTE = MFS_SEARCH_ANY;
-    search.WILDCARD = cur->name;
+    search.WILDCARD = name;
     search.SEARCH_DATA_PTR = &search_data;
 
     err = ioctl(mfs_ptr, IO_IOCTL_FIND_FIRST_FILE, (uint32_t*)&search);
@@ -4826,22 +4814,15 @@ static int SFTP_GetAttributes(void* fs, const char* fileName,
 }
 
 static int SFTP_GetAttributes_Handle(WOLFSSH* ssh, byte* handle, int handleSz,
-        WS_SFTP_FILEATRB* atr)
+        char* name, WS_SFTP_FILEATRB* atr)
 {
     FILINFO info;
-    WS_HANDLE_LIST *cur;
 
     if (handle == NULL || atr == NULL) {
         return WS_FATAL_ERROR;
     }
 
-    cur = SFTP_GetHandleNode(ssh, handle, handleSz);
-    if (cur == NULL) {
-        WLOG(WS_LOG_SFTP, "Unknown handle");
-        return WS_BAD_FILE_E;
-    }
-
-    if (f_stat(cur->name, &info) != FR_OK) {
+    if (f_stat(name, &info) != FR_OK) {
         return WS_FATAL_ERROR;
     }
 
@@ -4916,12 +4897,13 @@ int SFTP_GetAttributes(void* fs, const char* fileName, WS_SFTP_FILEATRB* atr,
 }
 
 int SFTP_GetAttributes_Handle(WOLFSSH* ssh, byte* handle, int handleSz,
-        WS_SFTP_FILEATRB* atr)
+        char* name, WS_SFTP_FILEATRB* atr)
 {
     WOLFSSH_UNUSED(ssh);
     WOLFSSH_UNUSED(handle);
     WOLFSSH_UNUSED(handleSz);
     WOLFSSH_UNUSED(atr);
+    WOLFSSH_UNUSED(name);
 
     WLOG(WS_LOG_SFTP, "SFTP_GetAttributes_Handle() not implemented yet");
     return WS_NOT_COMPILED;
@@ -5002,7 +4984,7 @@ int SFTP_GetAttributes(void* fs, const char* fileName, WS_SFTP_FILEATRB* atr,
 {
     WOLFSSH_UNUSED(heap);
     WOLFSSH_UNUSED(fs);
-    
+
     return SFTP_GetAttributesHelper(atr, fileName);
 }
 
@@ -5014,26 +4996,11 @@ int SFTP_GetAttributes(void* fs, const char* fileName, WS_SFTP_FILEATRB* atr,
  * returns WS_SUCCESS on success
  */
 int SFTP_GetAttributes_Handle(WOLFSSH* ssh, byte* handle, int handleSz,
-        WS_SFTP_FILEATRB* atr)
+        char* name, WS_SFTP_FILEATRB* atr)
 {
-    WS_HANDLE_LIST* cur;
-
-    if (handleSz != sizeof(word32)) {
-        WLOG(WS_LOG_SFTP, "Unexpected handle size SFTP_GetAttributes_Handle()");
-    }
-
-    cur = SFTP_GetHandleNode(ssh, handle, handleSz);
-    if (cur == NULL) {
-        WLOG(WS_LOG_SFTP, "Unknown handle");
-        return WS_BAD_FILE_E;
-    }
-        
     return SFTP_GetAttributesHelper(atr, cur->name);
 }
 
-#elif defined(WOLFSSH_USER_FILESYSTEM)
-    /* User-defined I/O support */
-
 #else
 
 /* NOTE: if atr->flags is set to a value of 0 then no attributes are set.
@@ -5143,6 +5110,7 @@ int wolfSSH_SFTP_RecvFSTAT(WOLFSSH* ssh, int reqId, byte* data, word32 maxSz)
     byte*  handle;
     word32 idx = 0;
     int ret = WS_SUCCESS;
+    char* name = NULL;
 
     byte*  out   = NULL;
     word32 outSz = 0;
@@ -5164,9 +5132,27 @@ int wolfSSH_SFTP_RecvFSTAT(WOLFSSH* ssh, int reqId, byte* data, word32 maxSz)
     }
     handle = data + idx;
 
+#ifdef WOLFSSH_STOREHANDLE
+    if (handleSz != sizeof(word32)) {
+        WLOG(WS_LOG_SFTP, "Unexpected handle size for stored handles");
+    }
+    else {
+        WS_HANDLE_LIST* cur;
+
+        cur = SFTP_GetHandleNode(ssh, handle, handleSz);
+
+        if (cur == NULL) {
+            WLOG(WS_LOG_SFTP, "Unknown handle");
+            return WS_BAD_FILE_E;
+        }
+        name = cur->name;
+    }
+#endif
+
     /* try to get file attributes and send back to client */
     WMEMSET((byte*)&atr, 0, sizeof(WS_SFTP_FILEATRB));
-    if (SFTP_GetAttributes_Handle(ssh, handle, handleSz, &atr) != WS_SUCCESS) {
+    if (SFTP_GetAttributes_Handle(ssh, handle, handleSz, name, &atr)
+            != WS_SUCCESS) {
         WLOG(WS_LOG_SFTP, "Unable to get fstat of file/directory");
         if (wolfSSH_SFTP_CreateStatus(ssh, WOLFSSH_FTP_FAILURE, reqId,
                 "STAT error", "English", NULL, &outSz) != WS_SIZE_ONLY) {