int cmdNumber(char* cmd) { char* cmd1; strcpy(cmd1, cmd); //Serial.println ("123"); char* ass = strsep(&cmd1, "M"); ass = strsep(&cmd1, " "); // Serial.println(ass); int aaa = atoi(ass); // Serial.println(aaa); return aaa; } void loop() { int v = cmdNumber("M1001 "); Serial.println(v, …

What is wrong with this code?

system November 17, 2012, 6:15pm 2

char* cmd1;
   strcpy(cmd1, cmd);

Dangerous.
Don't do it.

Topic		Replies	Views
String to char* conversion Programming Questions	36	6098	May 5, 2021
pointer and strings Programming Questions	10	8306	May 5, 2021
String question Troubleshooting	6	3017	May 6, 2021
Send Char* to Serial Programming Questions	10	1999	May 5, 2021
String ,char array and other vegetables Programming Questions	5	932	May 5, 2021