🚀 go-pugleaf

"robertbienert<AT>gmx.net gmx.net >" <"robertbienert<AT> wrote in message
news:3ef0da0c@newsgroups.borland.com...

> I'm very sorry, but I haven't checked GetLastError
> so far, maybe because it isn't mentioned in my
> version of Win32.HLP from www.activevb.de.

C++Builder comes with its own copy of the Win32 help files, and you can
always just look everything up at MSDN online directly.

> Here you are: That are the main c file plus the
> header file for both the dll and the using exe:

You didn't include the EXE's actual code at all.  You are not showing how
you are actually allocating the buffer that you want your DLL function to
fill in.  You're allowing any arbitrary buffer to be passed in from the EXE,
but the DLL is assuming its own buffer size instead of allowing the EXE to
tell the DLL exactly what the buffer size really is.

In fact, looking over this code, you are not assigning any kind of actual
value to your buffer size in the first place.  You declared your buffer size
as a pointer that you never actually assign to point to anywhere meaningful.
As such, you're going to have any sort of potential problems when you are
using the value later on because it is not valid to begin with.

I would suggest the simpler of the two possible solutions - just get rid of
your buffer size variable from the DLL altogether and let the function
accept an additional parameter so the EXE call specifically tell the DLL
exactly what the buffer size already is:

    int __stdcall getShellFolder(int folder, char *buffer, DWORD buffersize)


Gambit