I have a dll written in assembly and trying to make it c-callable, It works correctly except I don't think the stack is geting cleaned up properly. Does anyone know what I'm doing wrong? Or is it a macro problem? The test code is below, The executable is c code, and the dll is assembly. It uses my own custom includes, but the example is very small and should convert to standard fasm include format very easy. The code is below.

Your C program is assuming that your DLL proc is stdcall. Make your DLL stdcall too or try to fix your C environment. IMHO it's better to use stdcall in the DLL

Hey thanks! Changed "proc c ...." to "proc stdcall ..." and everything works.

[EDIT] Corrected version in the first post. [END EDIT]

Note that stdcall is the default so you can pass from "proc c ...." to "proc ...." if you want.

There is nothing wrong with having stdcall but if it's purely for C, C/cdecl is the default type for C functions and if you're compiler is assuming the functions are stdcall you have default calling convention set to stdcall. Anyways you should probably declare procedures with "type convention name( ... )" if it's in a Windows environment, even if they are cdecl.