problem in placing command line aruments(LPSTR) into **char

[Nagesh]

hi,

I am want copy the command line arguments from szCmdLine of
WinMain(HINSTAN CE hInstance, HINSTANCE hPrevInstance, PSTR szCmdLine,
int iCmdShow) into **char cmdLineParamete r. So that I can use
cmdLineParamete r in other functions. If any body knows pls help me

I am trying the following code sample:

char **cmdLineParame ter;
int k=0;
int WINAPI WinMain(HINSTAN CE hInstance, HINSTANCE hPrevInstance, PSTR
szCmdLine, int iCmdShow)
{
for (i = 0; i < strlen(szCmdLin e); i++)
{
// Now check for command-line arguments
if (strncmp(&szCmd Line[i], "-pw", strlen("-pw")) == 0)
{
cmdLineParamete r[k]=(char *)malloc(sizeof (char)*
strlen(&szCmdLi ne[i])));
strncpy(cmdLine Parameter[k],&szCmdLine[i],strlen(&szCmdL ine[i]));
k++;
continue;
}
if (strncmp(&szCmd Line[i], "-vpw", strlen("-vpw")) ==
0)
{
cmdLineParamete r[k]=(char *)malloc(sizeof (char)*
strlen(&szCmdLi ne[i])));
strncpy(cmdLine Parameter[k],&szCmdLine[i],strlen(&szCmdL ine[i]));
k++;
continue;
}
etc....
}
}
thanking u,
Nagesh

[pH]

The code you're using there will cause memory faults, as you never
allocate memory for the array cmdLineParamete rs itself, only for the
strings its elements point to; so, you need something like
cmdLineParamete r = (char **)malloc(sizeo f(char *) * something), where
something is the number of arguments you are expecting.

Also, using strncmp() at every offset into szCmdLine is rather
inefficient; it might be better to use strtok or similar to split the
string at each space (or whatever) character, and then put the results
in your cmdLineParamete r array. You'd still need to allocate memory for
this array; to avoid over/under allocating the amount, you might want
to count how many spaces appear in the szCmdLine and allocate space for
that many parameters, and then copy tokens out of the string into the
array.

[Nagesh]

hi,
I followed ur tips. I could run without any problem.

thank u very much,
Wish u a happy new year,
Nagesh