nihilist/tor
1
0
Fork 0
mirror of https://gitlab.torproject.org/tpo/core/tor.git synced 2024-11-13 14:43:46 +01:00
Code Issues Packages Projects Releases Wiki Activity

router: check for NULL in router_build_fresh_descriptor() static functions

Browse Source 
Make sure that these static functions aren't passed NULL.
If they are, log a BUG() warning, and return an error.

Preparation for testing 29017 and 20918.
This commit is contained in:
teor 2019-01-10 19:49:46 +10:00
parent f19b64dce9
commit a65c101973
1 changed files with 23 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

35
src/feature/relay/router.c
View File

@ -2080,6 +2080,7 @@ router_build_fresh_routerinfo(routerinfo_t **ri_out)
/** Allocate and return an extrainfo for this OR, based on the routerinfo ri. /** Allocate and return an extrainfo for this OR, based on the routerinfo ri.
* *
* If ri is NULL, logs a BUG() warning and returns NULL.
* Caller is responsible for freeing the generated extrainfo. * Caller is responsible for freeing the generated extrainfo.
*/ */
static extrainfo_t * static extrainfo_t *
@ -2087,6 +2088,9 @@ router_build_fresh_extrainfo(const routerinfo_t *ri)
{ {
extrainfo_t *ei = NULL; extrainfo_t *ei = NULL;
if (BUG(!ri))
return NULL;
/* Now generate the extrainfo. */ /* Now generate the extrainfo. */
ei = tor_malloc_zero(sizeof(extrainfo_t)); ei = tor_malloc_zero(sizeof(extrainfo_t));
ei->cache_info.is_extrainfo = 1; ei->cache_info.is_extrainfo = 1;
@ -2104,11 +2108,15 @@ router_build_fresh_extrainfo(const routerinfo_t *ri)
/** Create a signed descriptor for ei, and add it to ei->cache_info. /** Create a signed descriptor for ei, and add it to ei->cache_info.
* *
* Return 0 on success, -1 on temporary error. * Return 0 on success, -1 on temporary error.
* If ei is NULL, logs a BUG() warning and returns -1.
* On error, ei->cache_info is not modified. * On error, ei->cache_info is not modified.
*/ */
static int static int
router_update_extrainfo_descriptor_body(extrainfo_t *ei) router_update_extrainfo_descriptor_body(extrainfo_t *ei)
{ {
if (BUG(!ei))
return -1;
if (extrainfo_dump_to_string(&ei->cache_info.signed_descriptor_body, if (extrainfo_dump_to_string(&ei->cache_info.signed_descriptor_body,
ei, get_server_identity_key(), ei, get_server_identity_key(),
get_master_signing_keypair()) < 0) { get_master_signing_keypair()) < 0) {
@ -2129,23 +2137,27 @@ router_update_extrainfo_descriptor_body(extrainfo_t *ei)
} }
/** Set the fields in ri that depend on ei. /** Set the fields in ri that depend on ei.
*
* If ei is NULL, logs a BUG() warning and zeroes the relevant fields.
*/ */
static void static void
router_update_routerinfo_from_extrainfo(routerinfo_t *ri, router_update_routerinfo_from_extrainfo(routerinfo_t *ri,
const extrainfo_t *ei) const extrainfo_t *ei)
{ {
/* Now finish the router descriptor. */ if (BUG(!ei)) {
if (ei) { /* Just to be safe, zero ri->cache_info.extra_info_digest* here. */
memcpy(ri->cache_info.extra_info_digest, memset(ri->cache_info.extra_info_digest, 0, DIGEST_LEN);
ei->cache_info.signed_descriptor_digest, memset(ri->cache_info.extra_info_digest256, 0, DIGEST256_LEN);
DIGEST_LEN); return;
memcpy(ri->cache_info.extra_info_digest256,
ei->digest256,
DIGEST256_LEN);
} else {
/* ri was allocated with tor_malloc_zero, so there is no need to
* zero ri->cache_info.extra_info_digest here. */
} }
/* Now finish the router descriptor. */
memcpy(ri->cache_info.extra_info_digest,
ei->cache_info.signed_descriptor_digest,
DIGEST_LEN);
memcpy(ri->cache_info.extra_info_digest256,
ei->digest256,
DIGEST256_LEN);
} }
/** Create a signed descriptor for ri, and add it to ri->cache_info. /** Create a signed descriptor for ri, and add it to ri->cache_info.
@ -2261,7 +2273,6 @@ router_build_fresh_descriptor(routerinfo_t **r, extrainfo_t **e)
if (result < 0) if (result < 0)
goto skip_ei; goto skip_ei;
/* TODO: don't rely on tor_malloc_zero */
router_update_routerinfo_from_extrainfo(ri, ei); router_update_routerinfo_from_extrainfo(ri, ei);
/* TODO: disentangle these GOTOs, or split into another function. */ /* TODO: disentangle these GOTOs, or split into another function. */