C / C++ / MFC

Re: Stack overflow when calling one DLL function often?

14-May-03 1:23

Hello,

I have a problem with DLLs again Frown | :(

I have to load two DLLs which each contain 3 functions. The first function has to be called one time, the second very often, and the third only one time.

Now I call the second function about 60000 (each DLL) and I get a stack overflow???

The DLL function is:

DLL_C_EXPORT unsigned long gdPlay(unsigned long uRound, unsigned long uEnemyLast)
{
	return (unsigned long)(rand() % 2);
}

The calling loop in the main project is:

for(dwRound = 0; dwRound < m_dwRounds; dwRound++)
{
	dwCurrent1 = lpDllPlay1(dwRound, dwLast2);
	dwCurrent2 = lpDllPlay2(dwRound, dwLast1);
	if((dwCurrent1 == 1) && (dwCurrent2 == 1))
	{
		dwPoints += m_dwPoints11;
	}
	if((dwCurrent1 == 1) && (dwCurrent2 == 0))
	{
		dwPoints += m_dwPoints10;
	}
	if((dwCurrent1 == 0) && (dwCurrent2 == 1))
	{
		dwPoints += m_dwPoints01;
	}
	if((dwCurrent1 == 0) && (dwCurrent2 == 0))
	{
		dwPoints += m_dwPoints00;
	}
	dwLast1 = dwCurrent1; // Update variables
	dwLast2 = dwCurrent2;
}

The DLL is loaded perfectly and it also returns the correct values (I checked it with MessageBoxes)... But after some calls (about 60000 each) there is the stack overflow...

Do you have any idea what could cause the stack overflow??? Confused | :confused:

-Dominik

_outp(0x64, 0xAD);
and
__asm mov al, 0xAD __asm out 0x64, al
do the same... but what do they do??

Anonymous14-May-03 1:32

Anonymous

14-May-03 1:32

If the calling convention (stdcall etc) isn't correct in the call to the DLL function that might explain it. ie. who is responsible for popping params off of the stack.

Re: Stack overflow when calling one DLL function often?

Neville Franks14-May-03 1:34

Re: Stack overflow when calling one DLL function often?

14-May-03 1:34

I don't know why this came up as Anonymous - it was ME.

Neville Franks, Author of ED for Windows. Free Trial at www.getsoft.com

Dominik Reichl14-May-03 4:02

Re: Stack overflow when calling one DLL function often?

14-May-03 4:02

Hmm, how would that look?
Shall I put __stdcall or __cdecl before all functions?
In the main program I defined the function like this:

extern "C"
{
	typedef VOID(WINAPI *LPDLL_VOID)(VOID);
	typedef DWORD(WINAPI *LPDLL_2UL_1)(unsigned long uRound, unsigned long uEnemyLast);
	typedef DWORD(WINAPI *LPDLL_2UL_2)(unsigned long uRound, unsigned long uEnemyLast);
}

How would that look now?

Thanks for your help, you seem to be the DLL expert Big Grin | :-D

-Dominik

_outp(0x64, 0xAD);
and
__asm mov al, 0xAD __asm out 0x64, al
do the same... but what do they do??

Dominik Reichl14-May-03 4:09

Re: Stack overflow when calling one DLL function often?

14-May-03 4:09

Now it works!! Big Grin | :-D

I've replaced the WINAPI declaration (=__stdcall) with __cdecl and now everything works fine!

Thank you VERY much! Smile | :)

-Dominik

_outp(0x64, 0xAD);
and
__asm mov al, 0xAD __asm out 0x64, al
do the same... but what do they do??

Neville Franks14-May-03 11:10

Re: Stack overflow when calling one DLL function often?

14-May-03 11:10

Dominik Reichl wrote:
Now it works!! I've replaced the WINAPI declaration (=__stdcall) with __cdecl and now everything works fine!

Thank you VERY much!

Glad I was able to point you in the right direction. It would be interesting to know if this was related to your earlier problem of the DLL unloading and if putting the code optimization back how it was worked!

Neville Franks, Author of ED for Windows. Free Trial at www.getsoft.com

Dominik Reichl14-May-03 20:22

Re: Stack overflow when calling one DLL function often?

14-May-03 20:22

I enabled the code optimization again and tried it... the program doesn't crash any more with optimization turned on... so it seems these problems were related somehow. Dead | X|

You were right from the beginning, that code optimization cannot be the source of the problem... Smile | :)

-Dominik

_outp(0x64, 0xAD);
and
__asm mov al, 0xAD __asm out 0x64, al
do the same... but what do they do??

Neville Franks14-May-03 22:32

Caller ID compatible modems

14-May-03 22:32

Dominik Reichl wrote:
You were right from the beginning, that code optimization cannot be the source of the problem...

This really didn't sound right, which is why I nagged you a bit. Put it down to experience. Good to hear all the problems have been resolved. Rose | [Rose]

Neville Franks, Author of ED for Windows. Free Trial at www.getsoft.com

YSRao14-May-03 0:11

Re: Caller ID compatible modems

14-May-03 0:11

I am doing an application using TAPI32.lib.I have to capture the incoming call number.I have PCTel modem.AT#CID=1 command also working fine.I ahve down loaded caller ID applications and run,but they will not showing caller ID.My phone line has caller ID service.what is the caller ID system in India.
Anybody please kindly let me get rid of this problem.

yakkalas

Renjith Ramachandran15-May-03 5:59

Renjith Ramachandran

15-May-03 5:59

I am also searching for the same, if you get any solution please let me know also..

Thanks in advance

[ It is possible to represent everything in this universe by using 0 and 1 ]

Moving a file

dave_long14-May-03 0:09

dave_long

14-May-03 0:09

Hi what i want to accomplish is this:

To transfer a file from one folder to another folder without having to do that manually. ie. open both folders and drag and drop, or copy and paste.

What i want is to have a very basic console program or win based but preferably console based, that exists in the folder with the file and i just want to be able to drop the file on top of the console icon and for it to be sent to the destination folder.
I need almost no user input just to drop the file on top of the icon to run the program to send the file to its destination folder.

Anyone have anyi deas how i would go about doing this.
im using VC++.NET environ.

kind regards
Dave Long

Re: Moving a file

Neville Franks14-May-03 0:44

14-May-03 0:44

Have a look at WM_DROPFILES. I haven't done much with console apps, but this is what you use in a Windows app. Also CWnd::OnDropFiles(). This is what I use in ED (see sig) if the user drops a file on it, whether it is running or not. Hope that helps.

Neville Franks, Author of ED for Windows. Free Trial at www.getsoft.com

Re: Moving a file

Rage14-May-03 1:07

Rage

14-May-03 1:07

IMO you will need a winapp to do this.

A very easy way to do this is :

Make a batch file (like the following, just replace [destination path] with your path.)

@echo off

if "%1" == "" goto end
if not exist %1 goto end

copy %1 [destination path]

:end

Put the batch file (name it mycopy.bat) iy your windows SendTo directory.

Now every time you want to copy a file, right-click on it, go in the SendTo submenu and click on mycopy.bat. Your file will be copied. Cool | :cool:

~RaGE();

Convert DWORD to CString

JensB13-May-03 23:58

13-May-03 23:58

Hi
Need to convert a DWORD to a CString
and a CString to a DWORD type.

Hope someone knows Smile | :)

Greetings
Jens

YSRao14-May-03 0:41

14-May-03 0:41

Hi,
converting a CString into DWORD means, u are going to covert integer to char
here is example.

#define HELLO 100

DWORD dwrd=HELLO;
CString str1;
char str[10];
str1=itoa(HELLO,str,10);
//CString str1=str;
AfxMessageBox(str1);

this will convert the DWORD into CString,but it is not wise.You can't get CString value like "HELLO". it is not possible i think.you can get 100 as a CString value.

yakkalas

YSRao14-May-03 0:50

14-May-03 0:50

DWORD is also an unsigned integer,trying to convert it into CString means converting integer to char.
Here I am sending simple Ex.

DWORD dwrd=HELLO;
CString str1;
char str[10];
str1=::itoa(HELLO,str,10);
//CString str1=str;
AfxMessageBox(str1);

This simple code will show u 100 not HELLO.So as I am thinking,it is not possible.
You mean to try to get CString value as HELLO? Confused | :confused:

if so it is not possible as per my knowledge.

yakkalas.

YSRao14-May-03 0:51

14-May-03 0:51

DWORD is also an unsigned integer,trying to convert it into CString means converting integer to char.
Here I am sending simple Ex.

#define HELLO 100

DWORD dwrd=HELLO;
CString str1;
char str[10];
str1=::itoa(HELLO,str,10);
//CString str1=str;
AfxMessageBox(str1);

This simple code will show u 100 not HELLO.So as I am thinking,it is not possible.
You mean to try to get CString value as HELLO? Confused | :confused:

if so it is not possible as per my knowledge.

yakkalas.

JensB14-May-03 1:20

14-May-03 1:20

Well, it's like this i wanted:
I got a CString strValue = "5";

But i got a function which only accepts DWORD values;

so instead of CString strValue = "5";
It need to be converted to:
DWORD dvalue = 5;

that's all to it actually.

YSRao14-May-03 1:52

14-May-03 1:52

CString strValue="5";
DWORD dValue=atoi(strValue);
then dValue contains what is the value in strValue.
dValue=5;

yakkalas

Rage14-May-03 1:19

Rage

14-May-03 1:19

DWORD d;
CString str;

d=3;
str.Format("%i",d);

str="3";
d=atol(str.operator LPCTSSTR());

If this does not work, have a look at

http://www.codeproject.com/cpp/cppforumfaq.asp#cpp_atoi[^]

~RaGE();

JensB14-May-03 1:56

14-May-03 1:56

ok thanks for your help all.

I fixed the problem like this:

//CString -> long integer
CString strBurnr = m_listbur.GetItemText(1,2);
DWORD burnr = atol(strBurnr);

//Long Integer -> CString
char* szTemp;
ltoa(burnr, szTemp,10);
CString strTemp;
strTemp = (CString) szTemp;

Thanks, all works fine. Smile | :)

and btw (this isn't about conversions: does anyone of you guys know how to put a button in a field (in CListView) at every row?)

Greetings
Jens

David Crow14-May-03 3:01

David Crow

14-May-03 3:01

JensB wrote:
char* szTemp;
ltoa(burnr, szTemp,10);

This can't work. You've allocated no memory for szTemp. It's just a pointer pointing to who knows where!

To get a number, whether its an int, long, etc, into a CString object, just use CString::Format() as in:

DWORD dwNumber = 123;
CString strTemp;
strTemp.Format("%lu", dwNumber);

JensB14-May-03 3:55

14-May-03 3:55

yeah i'm sorry
I made a typo there
i had 'char szTemp[10];' here Smile | :)

Anyway, thanks Smile | :)