Some problems: You need to pass the /C option to cmd.exe in order to make it execute the .bat file. The second parameter to CreateProcess must be a modifiable string. Not a literal. You need to escape backslash characters in literals. lpszCurrentVariable points to the buffer returned by GetEnvironmentStrings. You cannot modify that buffer. You … Read more
The second argument is a LPTSTR, namely a pointer to a non-const char array. The docs specifically say: this parameter cannot be a pointer to read-only memory (such as a const variable or a literal string) The reason passing a string literal is a problem: The system adds a terminating null character to the command-line … Read more
You should specify also the module name in parameters: LPTSTR cmdArgs = “App [email protected]”; It should be the whole command line (including argv[0]).
Real problem: (from Wikipedia: http://en.wikipedia.org/wiki/User_Account_Control) An executable that is marked as “requireAdministrator” in its manifest cannot be started from a non-elevated process using CreateProcess(). Instead, ERROR_ELEVATION_REQUIRED will be returned. ShellExecute() or ShellExecuteEx() must be used instead. (BTW, ERROR_ELEVATION_REQUIRED error == 740) Solution: (same site) In a native Win32 application the same “runas” verb can be … Read more
Use ShellExecuteEx, rather than ShellExecute. This function will provide a handle for the created process, which you can use to call WaitForSingleObject on that handle to block until that process terminates. Finally, just call CloseHandle on the process handle to close it. Sample code (most of the error checking is omitted for clarity and brevity): … Read more