add rekey check to sftp client and use reletave path to file in test case

Author: JacobBarthelmeh

.github/workflows/windows-sftp.yml

@@ -318,16 +318,15 @@ jobs:
         }
         Write-Host "Basic SFTP test passed"
 
-    - name: Create 3GB test file and run SFTP get/put (large_rw)
-      if: matrix.test_type == 'large_rw'
+    - name: Create 3GB test file and run SFTP get/put
       working-directory: ${{ github.workspace }}\wolfssh
       shell: pwsh
       timeout-minutes: 25
       run: |
         $sftpPath = $env:SFTP_PATH
-        $workDir = (Get-Location).Path
+        $workDir = Join-Path $env:GITHUB_WORKSPACE "wolfssh"
         $largeFile = Join-Path $workDir "large_test.dat"
-        $getDest = Join-Path $workDir "large_test_copy.dat"
+        $getDestPath = Join-Path $workDir "large_test_copy.dat"
 
         # Create 3GB file: one random 10MB chunk repeated 307x + 2MB
         Write-Host "Creating 3GB test file..."
@@ -346,57 +345,61 @@ jobs:
         $fs.Close()
 
         $hash = Get-FileHash -Path $largeFile -Algorithm SHA256
-        $hash.Hash | Out-File -FilePath large_test.dat.sha256
+        $hash.Hash | Out-File -FilePath (Join-Path $workDir "large_test.dat.sha256")
         Write-Host "Created 3GB file, SHA256: $($hash.Hash)"
 
         # SFTP PUT (upload)
         Write-Host "SFTP PUT 3GB file..."
         $putCommands = "put $largeFile /large_test.dat`nquit"
-        $putCommands | Out-File -FilePath sftp_put_commands.txt -Encoding ASCII
+        $putCommands | Out-File -FilePath (Join-Path $workDir "sftp_put_commands.txt") -Encoding ASCII
         $proc = Start-Process -FilePath $sftpPath `
           -ArgumentList "-u", "testuser", "-P", $env:TESTUSER_PASSWORD, "-h", "localhost", "-p", "${{env.TEST_PORT}}" `
-          -RedirectStandardInput "sftp_put_commands.txt" `
-          -RedirectStandardOutput "sftp_put_out.txt" `
-          -RedirectStandardError "sftp_put_err.txt" `
+          -WorkingDirectory $workDir `
+          -RedirectStandardInput (Join-Path $workDir "sftp_put_commands.txt") `
+          -RedirectStandardOutput (Join-Path $workDir "sftp_put_out.txt") `
+          -RedirectStandardError (Join-Path $workDir "sftp_put_err.txt") `
           -Wait -NoNewWindow -PassThru
 
         if ($proc.ExitCode -ne 0) {
-          Get-Content sftp_put_out.txt
-          Get-Content sftp_put_err.txt
+          Get-Content (Join-Path $workDir "sftp_put_out.txt")
+          Get-Content (Join-Path $workDir "sftp_put_err.txt")
           Write-Host "ERROR: SFTP PUT failed"
           exit 1
         }
         Write-Host "PUT succeeded"
 
-        # SFTP GET (download)
+        # SFTP GET (download) - use full path so file lands in known location
+        # Use forward slashes for SFTP client (avoids backslash parsing in command)
         Write-Host "SFTP GET 3GB file..."
-        $getCommands = "get /large_test.dat $getDest`nquit"
-        $getCommands | Out-File -FilePath sftp_get_commands.txt -Encoding ASCII
+        $getDestPathSftp = $getDestPath -replace '\\', '/'
+        $getCommands = "get /large_test.dat $getDestPathSftp`nquit"
+        $getCommands | Out-File -FilePath (Join-Path $workDir "sftp_get_commands.txt") -Encoding ASCII
         $proc2 = Start-Process -FilePath $sftpPath `
           -ArgumentList "-u", "testuser", "-P", $env:TESTUSER_PASSWORD, "-h", "localhost", "-p", "${{env.TEST_PORT}}" `
-          -RedirectStandardInput "sftp_get_commands.txt" `
-          -RedirectStandardOutput "sftp_get_out.txt" `
-          -RedirectStandardError "sftp_get_err.txt" `
+          -WorkingDirectory $workDir `
+          -RedirectStandardInput (Join-Path $workDir "sftp_get_commands.txt") `
+          -RedirectStandardOutput (Join-Path $workDir "sftp_get_out.txt") `
+          -RedirectStandardError (Join-Path $workDir "sftp_get_err.txt") `
           -Wait -NoNewWindow -PassThru
 
         if ($proc2.ExitCode -ne 0) {
-          Get-Content sftp_get_out.txt
-          Get-Content sftp_get_err.txt
+          Get-Content (Join-Path $workDir "sftp_get_out.txt")
+          Get-Content (Join-Path $workDir "sftp_get_err.txt")
           Write-Host "ERROR: SFTP GET failed"
           exit 1
         }
         Write-Host "GET succeeded"
 
-        # Verify integrity
-        $expectedHash = (Get-Content large_test.dat.sha256).Trim()
-        $actualHash = (Get-FileHash -Path $getDest -Algorithm SHA256).Hash
+        # Verify integrity (file is in $workDir from GET with relative path)
+        $expectedHash = (Get-Content (Join-Path $workDir "large_test.dat.sha256")).Trim()
+        $actualHash = (Get-FileHash -Path $getDestPath -Algorithm SHA256).Hash
         if ($expectedHash -ne $actualHash) {
           Write-Host "ERROR: SHA256 mismatch - PUT/GET corruption"
           Write-Host "Expected: $expectedHash"
           Write-Host "Actual:   $actualHash"
           exit 1
         }
-        Write-Host "PASS: 3GB SFTP get/put with WOLFSSH_MAX_SFTP_RW=10485760 succeeded"
+        Write-Host "PASS: 3GB SFTP get/put succeeded"
 
     - name: Cleanup
       if: always()

examples/sftpclient/sftpclient.c

@@ -769,8 +769,8 @@ static int doCmds(func_args* args)
 
                 ret = wolfSSH_SFTP_STAT(ssh, pt, &atrb);
                 err = wolfSSH_get_error(ssh);
-            } while ((err == WS_WANT_READ || err == WS_WANT_WRITE)
-                        && ret != WS_SUCCESS);
+            } while ((err == WS_WANT_READ || err == WS_WANT_WRITE ||
+                        err == WS_REKEYING) && ret != WS_SUCCESS);
             if (ret != WS_SUCCESS) {
                 if (SFTP_FPUTS(args, "Error changing directory\n") < 0) {
                     err_msg("fputs error");
@@ -857,8 +857,8 @@ static int doCmds(func_args* args)
 
                 ret = wolfSSH_SFTP_CHMOD(ssh, pt, mode);
                 err = wolfSSH_get_error(ssh);
-            } while ((err == WS_WANT_READ || err == WS_WANT_WRITE)
-                        && ret != WS_SUCCESS);
+            } while ((err == WS_WANT_READ || err == WS_WANT_WRITE ||
+                        err == WS_REKEYING) && ret != WS_SUCCESS);
             if (ret != WS_SUCCESS) {
                 if (SFTP_FPUTS(args, "Unable to change permissions of ") < 0) {
                     err_msg("fputs error");
@@ -914,8 +914,8 @@ static int doCmds(func_args* args)
 
                 ret = wolfSSH_SFTP_RMDIR(ssh, pt);
                 err = wolfSSH_get_error(ssh);
-            } while ((err == WS_WANT_READ || err == WS_WANT_WRITE)
-                        && ret != WS_SUCCESS);
+            } while ((err == WS_WANT_READ || err == WS_WANT_WRITE ||
+                        err == WS_REKEYING) && ret != WS_SUCCESS);
             if (ret != WS_SUCCESS) {
                 if (ret == WS_PERMISSIONS) {
                     if (SFTP_FPUTS(args, "Insufficient permissions\n") < 0) {
@@ -967,8 +967,8 @@ static int doCmds(func_args* args)
 
                 ret = wolfSSH_SFTP_Remove(ssh, pt);
                 err = wolfSSH_get_error(ssh);
-            } while ((err == WS_WANT_READ || err == WS_WANT_WRITE)
-                        && ret != WS_SUCCESS);
+            } while ((err == WS_WANT_READ || err == WS_WANT_WRITE ||
+                        err == WS_REKEYING) && ret != WS_SUCCESS);
             if (ret != WS_SUCCESS) {
                 if (ret == WS_PERMISSIONS) {
                     if (SFTP_FPUTS(args, "Insufficient permissions\n") < 0) {