emo/ndhc
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix stupid typo in ndhc.c that would cause the clientid option to

Browse Source 
corrupt the start of the hostname option if both were specified.
This commit is contained in:
Nicholas J. Kain 2014-04-15 14:55:50 -04:00
parent 8a9fbb6f09
commit a777766cc6
1 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/ndhc.c
View File

@ -240,7 +240,7 @@ int get_clientid_string(char *str, size_t slen)
return -1; return -1;
if (!is_string_hwaddr(str, slen)) { if (!is_string_hwaddr(str, slen)) {
client_config.clientid[0] = 0; client_config.clientid[0] = 0;
memcpy(&client_config.clientid + 1, str, memcpy(client_config.clientid + 1, str,
min_size_t(slen, sizeof client_config.clientid - 1)); min_size_t(slen, sizeof client_config.clientid - 1));
client_config.clientid_len = slen + 1; client_config.clientid_len = slen + 1;
return 0; return 0;
@ -250,7 +250,8 @@ int get_clientid_string(char *str, size_t slen)
for (size_t i = 0; i < sizeof mac; ++i) for (size_t i = 0; i < sizeof mac; ++i)
mac[i] = strtol(str+i*3, NULL, 16); mac[i] = strtol(str+i*3, NULL, 16);
client_config.clientid[0] = 1; // Ethernet MAC type client_config.clientid[0] = 1; // Ethernet MAC type
memcpy(&client_config.clientid + 1, mac, sizeof mac); memcpy(client_config.clientid + 1, mac,
min_size_t(sizeof mac, sizeof client_config.clientid - 1));
client_config.clientid_len = 7; client_config.clientid_len = 7;
return 1; return 1;
} }