All it ever gives me is "Error: Invalid size of operand"

If i take out [FileName]/[CmdLine] and replace it with a NULL(s) the error goes away.
I've gone through any code i can find with CreateProcess but i just can't see what i'm doing wrong. I've been reading(yes i've searched these boards and google/iczelion's) and trying etc for half a day almost and it's now 3am so i though i'd post before i went to sleep and see if the fairy brings me anything in the morning. I am pretty new to Win32 mainly working with DOS so any help is appreciated. Chances are it's prolly something really simple knowing my luck and i just missed it :)

All i'm aiming todo is execute game.exe with some parameters but it's cause me more of a headache than i expected :)

Example snippet:


So, any help appreciated, known working code that i can just modify a dream :)


Thanks in advance,
Intirnsic

take off square brackets [] from FileName and CmdLine

Well, it compiles now thanks( i thought i had tried that already but obviously not :). But crashes so i gotta fix that now :)

So when "push FileName" happens it puts the address onto the stack so i don't need the [ ] correct?

Thanks Comrade

Yes, push FileName puts address of FileName on stack (equivalent to using OFFSET in MASM).
[FileName] is to get value of FileName, you would use that is FileName did not contain actual string, but contained pointer to that string

Thanks Comrade, i assumed it would work like mov si, [Buffer] for example but different with pushes i guess :) i had only ever pushed registers before.

Well, WinExec suited what i wanted todo more so used that instead in the end :) and it's 10 billion times easier.

WinExec is obsolete... If you are making a trainer, CreateProcess() already give you hProcess in PROCESSINFO structure.

No not a trainer, just a launcher for a game that refused to work with CreateProcess, but WinExec worked 1st time. This is a game that can only be launched by a dll of a certain name so i created a fake one that bypasses the patch server so you can point it at different server.
Ie, the way it was being done was to rename cmd.exe to login.dll then you would call the real dll with login.dll /C game.dll [parameters]
But this wasn't much use to Win98 users, and as the original launcher was written in VB it required a load of OCXs to resolve IP addresses etc. So I set out to create one that would work for Win98 users and didn't require VBloat components etc.
So, atm it calls it fine now and you launch the game and logon to the server, but it still gives me "login.dll has encountered a problem and needs to close" so i need to get rid of that.

Edit: i will add that the problem message disappears after i exit the game, but it's still annoying anyways :)