Thread: re-post garbled output: GetLogical DriveStrings

psDriveBuffer has to be an array, orelse you'll get an "access violation".
Try it this way:

//----------code-start--------------
char psDriveBuffer[256]; //modify array size as you need
DWORD nBufLen;

DWORD nRes;
nBufLen = 256;
printf("Getting Logical Names\n");

nRes = GetLogicalDriveStrings (nBufLen, &psDriveBuffer[0]);

...
//----------code-ends---------------

Have a nice code!

DWORD GetLogicalDriveStrings(
DWORD nBufferLength, // size of buffer
LPTSTR lpBuffer // pointer to buffer for drive strings
);

nBufferLength
Specifies the maximum size, in characters, of the buffer pointed to by lpBuffer. This size does not include the terminating null character.

lpBuffer
Pointer to a buffer that receives a series of null-terminated strings, one for each valid drive in the system, that end with a second null character. The following example shows the buffer contents with <null> representing the terminating null character.
c:\<null>d:\<null><null>



If you set the two parameters accordingly, i.e. allocated space
for the char* and wrote the size of the allocated space to
your dword, I don't see why you shouldn't get the first
logical drive. You won't get all, though with your method
of printing, because printf will go for zero terminated strings,
and your output has zeros in it that are not meant to be
the end of the output, just the end of one string.

To list the drives with printf (as each entry is 4 bytes long e.g. A:\terminating_zero) try this:

#define MAX_DRIVE_NO 24

char psDriveBuffer[MAX_DRIVE_NO][4];
...
GetLogicalDriveStrings (nBufLen, &psDriveBuffer[0][0]);

for ( int i = 0; i <= MAX_DRIVE_NO, i++ )
{
printf(%s"psDriveBuffer[i][0]);
}