Code:string cmdline = "pscp.exe -r -batch -scp -pw \""+m_password+"\" \""+m_file+"\" \""+m_username+"\"@\""+m_host+"\":\""+m_remotedir+"\""; STARTUPINFO si = { sizeof(si) }; PROCESS_INFORMATION pi; char *szExe = (char *)cmdline.c_str(); if(CreateProcess(0, (LPSTR)szExe, 0, 0, FALSE, 0, 0, 0, &si, &pi)) { WaitForSingleObject(pi.hProcess, INFINITE); CloseHandle(pi.hProcess); CloseHandle(pi.hThread); }
Thanks for your guys' help.
> char *szExe = (char *)cmdline.c_str();
This is dangerous, casting away the const-ness of an object.
Especially since CreateProcess() indicates that it can be modified.
If you dance barefoot on the broken glass of undefined behaviour, you've got to expect the occasional cut.
If at first you don't succeed, try writing your phone number on the exam paper.
