[dice7]

Why are you using strtok inside dcmd ? The whole point of dcmd is not using slow and inefficient strtok in the first place.
And the errors are that the variables 'player1' and 'string' are not defined