Using RasEnumConnections

[akia]

Could someone please assist me with this code, I have tried to create this from the help file and even though it compliles correctly, the RasEnumConnections API call will not return the correct results. My code is as follows.

{
int ReturnCode =0;
unsigned long *lpcb =0, *lpcConnections =0;
unsigned long *my_hRasConn =0;
RASCONN *lpRasConn[255];

lpRasConn[0]->dwSize = sizeof(RASCONN);

ReturnCode = RasEnumConnections(lpRasConn[0], lpcb, lpcConnections);

if(ReturnCode == ERROR_SUCCESS)
{
RasHangUp(lpRasConn);
}
my_hRasConn = my_hRasConn;
}

Thankyou

[CornedBee]

MSDN: (does nobody EVER take a look at it?)

Return Values
If the function succeeds, the return value is zero.

If the function fails, the return value is a nonzero error value listed in the RAS header file or one of ERROR_BUFFER_TOO_SMALL or ERROR_NOT_ENOUGH_MEMORY.

[Chris]

try this :)


LPRASENTRYNAME lpRasEntryName;
//
DWORD cEntries=0;
DWORD cb=0;
//
long idx=0;

// Create new RASENTRYNAME structure
lpRasEntryName = (LPRASENTRYNAME)GlobalAlloc(GPTR, sizeof(RASENTRYNAME));

// Initialize the RASENTRYNAME structure
lpRasEntryName->dwSize = sizeof(RASENTRYNAME);
// Get the entry name from registry
idx = RasEnumEntries(NULL, NULL, lpRasEntryName, &cb, &cEntries);
// Out of memory + relocate the RASENTRYNAME structure size
if (idx == ERROR_BUFFER_TOO_SMALL)
{
lpRasEntryName = (LPRASENTRYNAME)GlobalAlloc(GPTR, cb);
lpRasEntryName->dwSize = sizeof(RASENTRYNAME);
}

// Calling RasEnumEntries to enumerate the phone-book entries
idx = RasEnumEntries(NULL, NULL, lpRasEntryName, &cb, &cEntries);
// Error checking
if (idx != ERROR_SUCCESS)
{
// Notify user about the failure.
MessageBox(hWnd, TEXT("Fail to load the RasDial entry from registry!"), TEXT("MyRasDial Error"), MB_OK | MB_ICONEXCLAMATION);
// Set the return value
return 0;
}
else
{
// Loop through the return RASENTRYNAME structure
for(idx=0; idx < (long)cEntries; idx++)
{
// Add the ras entry into the comboboz control
SendDlgItemMessage(hWnd, IDC_COMBO1, CB_INSERTSTRING, -1, (LPARAM)lpRasEntryName->szEntryName);
lpRasEntryName++;
}
}
// Set the return value
return (long)cEntries;

[CornedBee]

GlobalAlloc is only kept for compatibility with 16-bit windows. You should use HeapAlloc instead.

[Chris]

I'll try out with HeapAlloc later :)