CID 1444117, CID 1444118 resource leaks in test_shared_random.c

** CID 1444117:    (RESOURCE_LEAK)
/src/test/test_shared_random.c: 1098 in test_utils_general()
/src/test/test_shared_random.c: 1099 in test_utils_general()


________________________________________________________________________________________________________
*** CID 1444117:    (RESOURCE_LEAK)
/src/test/test_shared_random.c: 1098 in test_utils_general()
1092           "1BDB7C3E973936E4D13A49F37C859B3DC69C429334CF9412E3FEF6399C52D47A";
1093         srv = tor_malloc_zero(sizeof(*srv));
1094         srv->num_reveals = 42;
1095         memcpy(srv->value, srv_value, sizeof(srv->value));
1096         dup_srv = sr_srv_dup(srv);
1097         tt_assert(dup_srv);
>>>     CID 1444117:    (RESOURCE_LEAK)
>>>     Variable "dup_srv" going out of scope leaks the storage it points to.
1098         tt_u64_op(dup_srv->num_reveals, OP_EQ, srv->num_reveals);
1099         tt_mem_op(dup_srv->value, OP_EQ, srv->value, sizeof(srv->value));
1100         tor_free(srv);
1101         tor_free(dup_srv);
1102       }
1103     
/src/test/test_shared_random.c: 1099 in test_utils_general()
1093         srv = tor_malloc_zero(sizeof(*srv));
1094         srv->num_reveals = 42;
1095         memcpy(srv->value, srv_value, sizeof(srv->value));
1096         dup_srv = sr_srv_dup(srv);
1097         tt_assert(dup_srv);
1098         tt_u64_op(dup_srv->num_reveals, OP_EQ, srv->num_reveals);
>>>     CID 1444117:    (RESOURCE_LEAK)
>>>     Variable "dup_srv" going out of scope leaks the storage it points to.
1099         tt_mem_op(dup_srv->value, OP_EQ, srv->value, sizeof(srv->value));
1100         tor_free(srv);
1101         tor_free(dup_srv);
1102       }
1103     
1104       /* Testing commitments_are_the_same(). Currently, the check is to test the

** CID 1444118:    (RESOURCE_LEAK)
/src/test/test_shared_random.c: 1098 in test_utils_general()
/src/test/test_shared_random.c: 1099 in test_utils_general()


________________________________________________________________________________________________________
*** CID 1444118:    (RESOURCE_LEAK)
/src/test/test_shared_random.c: 1098 in test_utils_general()
1092           "1BDB7C3E973936E4D13A49F37C859B3DC69C429334CF9412E3FEF6399C52D47A";
1093         srv = tor_malloc_zero(sizeof(*srv));
1094         srv->num_reveals = 42;
1095         memcpy(srv->value, srv_value, sizeof(srv->value));
1096         dup_srv = sr_srv_dup(srv);
1097         tt_assert(dup_srv);
>>>     CID 1444118:    (RESOURCE_LEAK)
>>>     Variable "srv" going out of scope leaks the storage it points to.
1098         tt_u64_op(dup_srv->num_reveals, OP_EQ, srv->num_reveals);
1099         tt_mem_op(dup_srv->value, OP_EQ, srv->value, sizeof(srv->value));
1100         tor_free(srv);
1101         tor_free(dup_srv);
1102       }
1103     
/src/test/test_shared_random.c: 1099 in test_utils_general()
1093         srv = tor_malloc_zero(sizeof(*srv));
1094         srv->num_reveals = 42;
1095         memcpy(srv->value, srv_value, sizeof(srv->value));
1096         dup_srv = sr_srv_dup(srv);
1097         tt_assert(dup_srv);
1098         tt_u64_op(dup_srv->num_reveals, OP_EQ, srv->num_reveals);
>>>     CID 1444118:    (RESOURCE_LEAK)
>>>     Variable "srv" going out of scope leaks the storage it points to.
1099         tt_mem_op(dup_srv->value, OP_EQ, srv->value, sizeof(srv->value));
1100         tor_free(srv);
1101         tor_free(dup_srv);
1102       }
1103     
1104       /* Testing commitments_are_the_same(). Currently, the check is to test the

These look to be only leaks in assertion-failure cases. Maybe make the pointers in question function-scope, or break stuff out into helper functions so the done label can properly clean up?