Fix possible unintended negative loop bound

This commit is contained in:
jfreegman 2022-01-01 13:47:35 -05:00
parent e6d671eeab
commit 53bd3c86df
No known key found for this signature in database
GPG Key ID: 3627F3144076AE63

View File

@ -443,7 +443,12 @@ int groupchat_enable_av(const Logger *log, Tox *tox, Group_Chats *g_c, uint32_t
return -1; return -1;
} }
int numpeers = group_number_peers(g_c, groupnumber, false); const int numpeers = group_number_peers(g_c, groupnumber, false);
if (numpeers < 0) {
kill_group_av(group_av);
return -1;
}
for (uint32_t i = 0; i < numpeers; ++i) { for (uint32_t i = 0; i < numpeers; ++i) {
group_av_peer_new(group_av, groupnumber, i); group_av_peer_new(group_av, groupnumber, i);
@ -470,7 +475,12 @@ int groupchat_disable_av(Group_Chats *g_c, uint32_t groupnumber)
return -1; return -1;
} }
int numpeers = group_number_peers(g_c, groupnumber, false); const int numpeers = group_number_peers(g_c, groupnumber, false);
if (numpeers < 0) {
kill_group_av(group_av);
return -1;
}
for (uint32_t i = 0; i < numpeers; ++i) { for (uint32_t i = 0; i < numpeers; ++i) {
group_av_peer_delete(group_av, groupnumber, group_peer_get_object(g_c, groupnumber, i)); group_av_peer_delete(group_av, groupnumber, group_peer_get_object(g_c, groupnumber, i));