evetestkit : sanity check in AppSCPCopy

Do sanity check in AppSCPCopy to make sure that the source does exist and it is not a directory. Signed-off-by: Shahriyar Jalayeri <[email protected]>
lf-edge · Oct 9, 2024 · ccd4f3c · ccd4f3c
1 parent 6b9d7ce
commit ccd4f3c
Showing 1 changed file with 8 additions and 0 deletions.
diff --git a/pkg/evetestkit/utils.go b/pkg/evetestkit/utils.go
@@ -291,6 +291,14 @@ func (node *EveNode) AppSSHExec(appName, command string) (string, error) {
 
 // AppSCPCopy copies a file from the local machine to the app VM running on the EVE node.
 func (node *EveNode) AppSCPCopy(appName, localFile, remoteFile string) error {
+	info, err := os.Stat(localFile)
+	if os.IsNotExist(err) {
+		return fmt.Errorf("file %s does not exist", localFile)
+	}
+	if info.IsDir() {
+		return fmt.Errorf("file %s is a directory", localFile)
+	}
+
 	appConfig := node.getAppConfig(appName)
 	if appConfig == nil {
 		return fmt.Errorf("app %s not found, make sure to deploy app/vm with WithSSH option", appName)