Fix minor issues and Add test vectors

Author: yosuke-wolfssl

apps/wolfsshd/configuration.c

@@ -103,13 +103,17 @@ static long GetConfigInt(const char* in, int inSz, int isTime, void* heap)
     int mult = 1; /* multiplier */
     int sz   = inSz;
 
+    if (in == NULL || inSz <= 0) {
+        ret = WS_BAD_ARGUMENT;
+    }
+
     /* check for multipliers */
-    if (isTime) {
+    if (ret == 0 && isTime) {
         if (in[sz - 1] == 'm') {
             sz--;
             mult = 60;
         }
-        if (in[sz - 1] == 'h') {
+        else if (in[sz - 1] == 'h') {
             sz--;
             mult = 60*60;
         }

apps/wolfsshd/test/test_configuration.c

@@ -229,6 +229,8 @@ static int test_ParseConfigLine(void)
         {"Valid login grace time minutes", "LoginGraceTime 1m", 0},
         {"Valid login grace time hours", "LoginGraceTime 1h", 0},
         {"Invalid login grace time", "LoginGraceTime wolfsshd", 1},
+        {"Bare multiplier m (no digit)", "LoginGraceTime m", 1},
+        {"Bare multiplier h (no digit)", "LoginGraceTime h", 1},
 
         /* Permit empty password tests. */
         {"Permit empty password no", "PermitEmptyPasswords no", 0},

examples/scpclient/scpclient.c

@@ -296,9 +296,6 @@ THREAD_RETURN WOLFSSH_THREAD scp_client(void* args)
     if (ret != WS_SUCCESS)
         err_sys("Couldn't set the session's socket.");
 
-    if (ret != WS_SUCCESS)
-        err_sys("Couldn't set the channel type.");
-
     do {
         if (dir == copyFromSrv)
             ret = wolfSSH_SCP_from(ssh, path1, path2);

src/wolfsftp.c

@@ -1835,7 +1835,7 @@ int wolfSSH_SFTP_RecvRMDIR(WOLFSSH* ssh, int reqId, byte* data, word32 maxSz)
  */
 int wolfSSH_SFTP_RecvMKDIR(WOLFSSH* ssh, int reqId, byte* data, word32 maxSz)
 {
-    word32 attrFlags;
+    WS_SFTP_FILEATRB atr;
     word32 strSz;
     const byte* str;
     int    ret;
@@ -1866,18 +1866,16 @@ int wolfSSH_SFTP_RecvMKDIR(WOLFSSH* ssh, int reqId, byte* data, word32 maxSz)
         return ret;
     }
 
-    if (GetUint32(&attrFlags, data, maxSz, &idx) != WS_SUCCESS) {
+    if (SFTP_ParseAttributes_buffer(ssh, &atr, data, &idx, maxSz)
+            != WS_SUCCESS) {
         return WS_BUFFER_E;
     }
-    if (attrFlags != WOLFSSH_FILEATRB_PERM) {
-        WLOG(WS_LOG_SFTP, "Only permission attribute supported");
-        WLOG(WS_LOG_SFTP, "Skipping over attribute and using default");
-        mode = 040755;
+    if (atr.flags & WOLFSSH_FILEATRB_PERM) {
+        mode = atr.per;
     }
     else {
-        if (GetUint32(&mode, data, maxSz, &idx) != WS_SUCCESS) {
-            return WS_BUFFER_E;
-        }
+        WLOG(WS_LOG_SFTP, "No permission attribute, using default");
+        mode = 040755;
     }
 
 #ifndef USE_WINDOWS_API