mirror of
https://gitlab.torproject.org/tpo/core/tor.git
synced 2024-11-10 21:23:58 +01:00
r18294@catbus: nickm | 2008-02-20 22:42:44 -0500
Fix a spelling error and clean up a recent veracode-induced integer overflow check. Both spotted by Chris Palmer. svn:r13639
This commit is contained in:
parent
1df0647c66
commit
47e6247673
@ -89,10 +89,12 @@ smartlist_add(smartlist_t *sl, void *element)
|
||||
void
|
||||
smartlist_add_all(smartlist_t *s1, const smartlist_t *s2)
|
||||
{
|
||||
smartlist_ensure_capacity(s1, s1->num_used + s2->num_used);
|
||||
tor_assert(s1->capacity >= s1->num_used+s2->num_used);
|
||||
int new_size = s1->num_used + s2->num_used;
|
||||
tor_assert(new_size >= s1->num_used); /* check for overflow. */
|
||||
smartlist_ensure_capacity(s1, new_size);
|
||||
tor_assert(s1->capacity >= new_size);
|
||||
memcpy(s1->list + s1->num_used, s2->list, s2->num_used*sizeof(void*));
|
||||
s1->num_used += s2->num_used;
|
||||
s1->num_used = new_size;
|
||||
}
|
||||
|
||||
/** Remove all elements E from sl such that E==element. Preserve
|
||||
|
@ -4754,7 +4754,7 @@ or_state_load(void)
|
||||
"to \"%s\". This could be a bug in Tor; please tell "
|
||||
"the developers.", fname, fname2);
|
||||
if (rename(fname, fname2) < 0) {
|
||||
log_warn(LD_BUG, "Weirdly, I couldn't even mode the state aside. The "
|
||||
log_warn(LD_BUG, "Weirdly, I couldn't even move the state aside. The "
|
||||
"OS gave an error of %s", strerror(errno));
|
||||
}
|
||||
}
|
||||
|
Loading…
Reference in New Issue
Block a user