No sure if this matters but I think the structures used in this code should be initialised to null before use, e.g.:

STARTUPINFO si;
PROCESS_INFORMATION pi;

ZeroMemory( &si, sizeof(si) );
si.cb = sizeof(si);
ZeroMemory( &pi, sizeof(pi) );

After some investigation I realised this isn't correct.