🚀 go-pugleaf

"Jeff" <Jeff@nospam.com> wrote in message
news:42c2e09e$1@newsgroups.borland.com...

> I'm trying to use a Windows API call to get the full name of
> the person logged into the current system. When I compile
> my code below I get an "Unresolved external 'GetUserNameExW'
> referenced from..." linker error.

Try adding "secur32.lib" to your project.  It is in the $(BCB)\Lib\PSDK
folder.

> WCHAR   wszDN[MAXBUFF];  //Character buffer for DN Name
> ULONG   cchDN = MAXBUFF; //Length of DN Name

Rather than hard-coding your buffer size, you should allow GetUserNameEx()
to tell you what buffer size to use.

    ULONG cchDN = 0;
    GetUserNameEx(NameDisplay, NULL, &cchDN);
    if( GetLastError() == ERROR_MORE_DATA )
    {
        WCHAR *wszDN = new WCHAR[cchDN];
        if( GetUserNameEx(NameDisplay, wszDN, &cchDN) )
            //...
        delete[] wszDN;
    }

> if (!GetUserNameEx(NameDisplay, wszDN, &cchDN))

Do note that GetUserNameEx() is only available on Win2K Pro/Server, XP, and
Win2K3 Server.  If you need your application to run on any other OS
versions, then you cannot statically link to GetUserNameEx() or else the
application will fail to run at all on them.  You would have to use
LoadLibrary() and GetProcAddress() instead to access the function pointer
dynamically at runtime.

>     lbeAddress->Text = static_cast<String>(wszDN);

That is a very dangerous cast.  You are casting a wchar_t[] buffer into an
AnsiString, which it is not.  Just assign the wchar_t[] directly to the Text
instead.  AnsiString has a constructor that accepts a wchar_t* as a
parameter:

    lbeAddress->Text = wszDN;

Or alternatively, it also has a constructor that accepts a WideString as a
parameter:

    lbeAddress->Text = WideString(wszDN, cchDN);


Gambit