j3d1/esp-open-rtos
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

Merge pull request #350 from funnydog/sntp-fix

Browse source 
extras/sntp: fix an off-by-one bug in sntp_set_servers()
This commit is contained in:
Ruslan V. Uss 2017-03-15 10:58:28 +06:00 committed by GitHub
commit 20674f8470
1 changed files with 2 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
extras/sntp/sntp.c
View file

@ -730,10 +730,8 @@ int sntp_set_servers(char *server_url[], int num_servers)
/* Allocate memory and copy servers */ /* Allocate memory and copy servers */
for (i = 0; i < num_servers; i++) { for (i = 0; i < num_servers; i++) {
sntp_server_addresses[i] = malloc(strlen(server_url[i])); sntp_server_addresses[i] = strdup(server_url[i]);
if (sntp_server_addresses[i]) { if (!sntp_server_addresses[i]) {
strcpy(sntp_server_addresses[i], server_url[i]);
} else {
sntp_num_servers = i; sntp_num_servers = i;
return -2; return -2;
} }