[VXP]: fix memory leak when hooking dex2oat

tiann · tiann · commit 09e6e037e1cf · 2018-06-16T20:54:16.000+08:00
diff --git a/VirtualApp/lib/src/main/jni/Foundation/IOUniformer.cpp b/VirtualApp/lib/src/main/jni/Foundation/IOUniformer.cpp
@@ -613,6 +613,8 @@ HOOK_DEF(int, execve, const char *pathname, char *argv[], char *const envp[]) {
         char **new_argv = build_new_argv(argv);
         int ret = syscall(__NR_execve, redirect_path, new_argv, new_envp);
         FREE(redirect_path, pathname);
+        free(new_envp);
+        free(new_argv);
         return ret;
     }
     int ret = syscall(__NR_execve, redirect_path, argv, envp);

Original file line number	Diff line number	Diff line change
`@@ -613,6 +613,8 @@ HOOK_DEF(int, execve, const char pathname, char argv[], char *const envp[]) {`
`613`	`613`	`char **new_argv = build_new_argv(argv);`
`614`	`614`	`int ret = syscall(__NR_execve, redirect_path, new_argv, new_envp);`
`615`	`615`	`FREE(redirect_path, pathname);`
	`616`	`+ free(new_envp);`
	`617`	`+ free(new_argv);`
`616`	`618`	`return ret;`
`617`	`619`	`}`
`618`	`620`	`int ret = syscall(__NR_execve, redirect_path, argv, envp);`