Fixed minor issues found with static code analyzis (Clang).
This commit is contained in:
parent
0ae8a08f55
commit
a9b5c6db38
|
@ -86,7 +86,8 @@ static int check_cmd_user(const char* cmd, int status, struct linked_list* list,
|
||||||
}
|
}
|
||||||
|
|
||||||
strncpy(info->nickname, data, MAX_NICK_LEN);
|
strncpy(info->nickname, data, MAX_NICK_LEN);
|
||||||
strncpy(info->password, data_extra, MAX_PASS_LEN);
|
if (data_extra)
|
||||||
|
strncpy(info->password, data_extra, MAX_PASS_LEN);
|
||||||
info->credentials = status;
|
info->credentials = status;
|
||||||
list_append(list, info);
|
list_append(list, info);
|
||||||
LOG_DEBUG("ACL: Added user '%s' (%s)", info->nickname, auth_cred_to_string(info->credentials));
|
LOG_DEBUG("ACL: Added user '%s' (%s)", info->nickname, auth_cred_to_string(info->credentials));
|
||||||
|
|
|
@ -203,8 +203,7 @@ int ip_mask_create_left(int af, int bits, struct ip_addr_encap* result)
|
||||||
fill = (128-bits) / 8;
|
fill = (128-bits) / 8;
|
||||||
remain_bits = (128-bits) % 8;
|
remain_bits = (128-bits) % 8;
|
||||||
mask = (0xff << (8 - remain_bits));
|
mask = (0xff << (8 - remain_bits));
|
||||||
n = 0;
|
|
||||||
|
|
||||||
for (n = 0; n < fill; n++)
|
for (n = 0; n < fill; n++)
|
||||||
((uint8_t*) &result->internal_ip_data.in6)[n] = (uint8_t) 0xff;
|
((uint8_t*) &result->internal_ip_data.in6)[n] = (uint8_t) 0xff;
|
||||||
|
|
||||||
|
@ -252,7 +251,6 @@ int ip_mask_create_right(int af, int bits, struct ip_addr_encap* result)
|
||||||
fill = (128-bits) / 8;
|
fill = (128-bits) / 8;
|
||||||
remain_bits = (128-bits) % 8;
|
remain_bits = (128-bits) % 8;
|
||||||
mask8 = (0xff >> (8 - remain_bits));
|
mask8 = (0xff >> (8 - remain_bits));
|
||||||
n = 0;
|
|
||||||
start = 16-fill;
|
start = 16-fill;
|
||||||
|
|
||||||
for (n = 0; n < start; n++)
|
for (n = 0; n < start; n++)
|
||||||
|
|
Loading…
Reference in New Issue