No, it should not. Strtok returns a NULL when no more tokens are found. On the first call it found "string to split up". The return will be a pointer to that token, which happens to be the entire input string.

Do yourself a favor: read the documentations for the things you use.

Do us a favor: learn to use the code tags like a big boy. Quit forcing the task on Shabbir.