Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

"A call to a PInvoke function has unbalanced the stack"

I created a Form application in visual c#, that uses a function to generate mouse click, but i got the following error message:

A call to PInvoke function '...Form1::mouse_event' has unbalanced the stack.
This is likely because the managed PInvoke signature does not match the unmanaged target
signature. Check that the calling convention and parameters of the PInvoke signature match 
the target unmanaged signature.

My Code:

[DllImport("user32.dll", CharSet = CharSet.Auto, CallingConvention = CallingConvention.StdCall)]
public static extern void mouse_event(long dwFlags, long dx, long dy, long cButtons, long dwExtraInfo);

private const int MOUSEEVENTF_LEFTDOWN = 0x02;
private const int MOUSEEVENTF_LEFTUP = 0x04;

...

void GenerateMouseClick(int x, int y)
{
    Cursor.Position = new Point((int)x, (int)y);
    mouse_event(MOUSEEVENTF_LEFTDOWN | MOUSEEVENTF_LEFTUP, Cursor.Position.X, Cursor.Position.Y, 0, 0);
}
like image 498
Pmillan Avatar asked Mar 24 '12 20:03

Pmillan


1 Answers

Your Win32 API declaration is incorrect: 'long' maps to Int64 in the .NET Framework, which is almost always incorrect for Windows API calls.

Replacing long with int should work:

public static extern void mouse_event(int dwFlags, int dx, int dy, int cButtons, int dwExtraInfo);

For future reference, you may want to check pinvoke.net whenever you're looking for the correct way to invoke API functions -- although it's not perfect, it would have shown the correct declaration for mouse_event.

(EDIT, 26 March 2012): And although the declaration I provided indeed works, replacing long with uint would be even better, as Win32's DWORD is a 32-bit unsigned integer. In this case, you'll get away with using a signed integer instead (as neither the flags nor the other arguments will ever be large enough to cause sign overflow), but this is definitely not always the case. The pinvoke.net declaration is correct, as is the following:

public static extern void mouse_event(uint dwFlags, uint dx, uint dy, uint cButtons, uint dwExtraInfo);

Another answer to this question already provided this correct declaration, and the uint issue was also pointed out in comments. I edited my own answer to make this more obvious; other SO participants should always feel free to edit incorrect posts as well, BTW.

like image 188
mdb Avatar answered Oct 07 '22 21:10

mdb