improvements to wolfSSH_CleanPath function used with SCP

[JacobBarthelmeh]

ZD20992

[Copilot]

Pull request overview

This PR improves the wolfSSH_CleanPath function by adding a buffer size parameter to prevent buffer overflow issues when processing file paths in SCP operations. The changes enhance security by validating that cleaned paths will fit in destination buffers before copying.

Key Changes:

• Added inSz parameter to wolfSSH_CleanPath to specify buffer size
• Updated buffer overflow check to verify cleaned paths fit before copying
• Updated all call sites to pass appropriate buffer sizes

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

File	Description
wolfssh/internal.h	Updated function signature to include buffer size parameter
src/internal.c	Implemented buffer size validation, improved buffer allocation (+2 bytes for delimiter and null terminator), enhanced error message, removed redundant code
src/wolfscp.c	Updated all call sites to pass appropriate buffer sizes (DEFAULT_SCP_MSG_SZ, DEFAULT_SCP_FILE_NAME_SZ, WOLFSSH_MAX_FILENAME, ssh->scpBasePathSz)

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.