Im trying to pass a pointer to a string to a dll loaded at runtime so that i can modify the value of the string from the dll however I am getting buffer overruns. If anyone could shed some light on what im doing wrong it would be appreciated.
http://www.dakotacom.net/~bthomas/dll.txt
>>sprintf(crc, "%x", test);
Look up the second arg here
it's not the sprintf call, i can take it out of the code completely and it still crashes...
ups
Stop bumping your thread and actually be useful for a change. Don't just say "it crashes" because that helps on one.
Quzah.
Hope is the first step on the road to disappointment.
the GetModuleFileName() could be expecting TCHAR's (that could be 2 wide if using UNICODE) and so using 100 + terminator bytes of your 50 byte buffer........
"Man alone suffers so excruciatingly in the world that he was compelled to invent laughter."
Friedrich Nietzsche
"I spent a lot of my money on booze, birds and fast cars......the rest I squandered."
George Best
"If you are going through hell....keep going."
Winston Churchill
I am pretty positive it has to do with passing the pointer because I can modify the function and pass ints just fun, but as soon as I try to pass a pointer I get the buffer overrun. I'm stumped.
Right...ok...I tried your code and there are a few nasties;
- #include <iostream.h> - in a c file! Dont do it, C++ only when using that header
- void (WINAPI*makeCRC)(char *crc); - no WINAPI declaration for a function you created yourself....if you made the function, it's using the C calling convention....WINAPI forces the stdcall convention...no doubt this is the cause of your crash
- void (WINAPI*makeCRC)(char *crc); - again this isnt what it's supposed to be....you need to use this line as part of a typedef, then use that to create the function pointer
so the func declaration in the exe will look like this
Code:typedef void (*MAKECRC)(char *crc); MAKECRC makeCRC;
Made that change and now im getting an error trying to compile :/
error C2440: '=' : cannot convert from 'FARPROC' to 'MAKECRC'
on line:
makeCRC=GetProcAddress((HMODULE)hLib, "makeCRC");
Then cast it
makeCRC=(MAKECRC)GetProcAddress(hLib, "makeCRC");
