Merge pull request #859 from anhu/offby1

ejohnstown · web-flow · commit b7c26d273a00 · 2025-12-30T13:05:25.000-07:00
Fix an off by one error.
diff --git a/src/internal.c b/src/internal.c
@@ -17007,7 +17007,7 @@ int wolfSSH_CleanPath(WOLFSSH* ssh, char* in)
     /* remove any /./ patterns, direcotries, exclude cases like ./ok./test */
     for (i = 1; i + 1 < sz; i++) {
         if (path[i] == '.' && path[i - 1] == WS_DELIM && path[i + 1] == WS_DELIM) {
-            WMEMMOVE(path + (i-1), path + (i+1), sz - (i-1));
+            WMEMMOVE(path + (i-1), path + (i+1), sz - i);
             sz -= 2; /* removed '/.' from string*/
             i--;
         }

Original file line number	Diff line number	Diff line change
`@@ -17007,7 +17007,7 @@ int wolfSSH_CleanPath(WOLFSSH* ssh, char* in)`
`17007`	`17007`	`/* remove any /./ patterns, direcotries, exclude cases like ./ok./test */`
`17008`	`17008`	`for (i = 1; i + 1 < sz; i++) {`
`17009`	`17009`	`if (path[i] == '.' && path[i - 1] == WS_DELIM && path[i + 1] == WS_DELIM) {`
`17010`		`- WMEMMOVE(path + (i-1), path + (i+1), sz - (i-1));`
	`17010`	`+ WMEMMOVE(path + (i-1), path + (i+1), sz - i);`
`17011`	`17011`	`sz -= 2; /* removed '/.' from string*/`
`17012`	`17012`	`i--;`
`17013`	`17013`	`}`