Rewrote NULL-termination code to avoid writing to a const array

This commit is contained in:
adamdunkels 2010-02-04 14:59:08 +00:00
parent 35f8d24c2e
commit 4ea2312f23

View file

@ -30,7 +30,7 @@
* *
* Author: Adam Dunkels <adam@sics.se> * Author: Adam Dunkels <adam@sics.se>
* *
* $Id: shell-udpsend.c,v 1.3 2010/02/04 11:20:09 adamdunkels Exp $ * $Id: shell-udpsend.c,v 1.4 2010/02/04 14:59:08 adamdunkels Exp $
*/ */
#include <string.h> #include <string.h>
@ -84,9 +84,9 @@ PROCESS_THREAD(shell_udpsend_process, ev, data)
"udpsend <server> <port> [localport]: server as address", ""); "udpsend <server> <port> [localport]: server as address", "");
PROCESS_EXIT(); PROCESS_EXIT();
} }
/* NULL-terminate the server string before copying. */
*next = 0;
strncpy(server, data, sizeof(server)); strncpy(server, data, sizeof(server));
/* NULL-terminate the server string. */
server[next - (char *)data] = 0;
++next; ++next;
port = shell_strtolong(next, &nextptr); port = shell_strtolong(next, &nextptr);