🚀 go-pugleaf

Hi,
I am working on a client-server project, and have a problem on the client 
side.
I execute commands on the client side with cmd.exe and send the output to 
server.
More clearly I first
-create pipes to redirect stdout and stdin of child process

-call CreateProcess to create the process, with pipes as stdin and out. (the 
process created is something like "c:\windows\system32\cmd.exe /C 
thecommand". path of cmd.exe is acquired from ComSpec environment variable.)

-start a thread to read the output of the child process.
   --This thread calls ReadFile and appends the data read to a 
std::vector<TCHAR*>. To syncronize access to the vector I used a mutex, no 
thread manipulates the vector without acquiring mutex.
   --I also handled (I guess but not sure) pending io case 
(ERROR_IO_PENDING) for ReadFile.

-main thread sends the data in the vector to the server using socket 
connection

My problem is; if I execute "dir" as command everything works fine. But for 
example if I execute a net command like "new view" I get access violation 
errors or I get various code guard  messages like reference to freed memory, 
invalid memory location etc while trying to send the data to the server.

I guess I am not handling reading of data from the pipe properly. In fact I 
didn't understand overlapped structure clearly, should I use it and if so 
how should I use it? Or should I use ReadFileEx, which also uses overlapped 
structure.

And one other question not directly related to the other question, how 
should I send unicode data using sockets? send accepts char* but not 
wchar_t*. Should I convert it using WideCharToMultiByte or what should I do? 
What is the best way of doing this?


Thanks in advance

(By the way I know I should send a simplified version of the code but I 
could not create a simplified piece of code that creates the same errors, 
because that would not be a simple and short piece of code.)