diff options
| author | Corey Farrell <git@cfware.com> | 2017-10-05 18:59:06 -0400 |
|---|---|---|
| committer | Corey Farrell <git@cfware.com> | 2017-10-06 14:38:58 -0400 |
| commit | 3bd00c4a7eeb9da143b5f7ff1ec12bdfcc2c5af7 (patch) | |
| tree | 9c1aae13293ecfceeaaa442d48c8ca1353b7dd4d /tests/test_vector.c | |
| parent | 3ef6834c3b6fb9cb73164dcc56a10820026a63c5 (diff) | |
vector: multiple evaluation of elem in AST_VECTOR_ADD_SORTED.
Use temporary variable to prevent multiple evaluations of elem argument.
This resolves a memory leak in res_pjproject startup.
ASTERISK-27317 #close
Change-Id: Ib960d7f5576f9e1a3c478ecb48995582a574e06d
Diffstat (limited to 'tests/test_vector.c')
| -rw-r--r-- | tests/test_vector.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tests/test_vector.c b/tests/test_vector.c index 8e0d121dd..2dfcc60a8 100644 --- a/tests/test_vector.c +++ b/tests/test_vector.c @@ -210,7 +210,7 @@ AST_TEST_DEFINE(basic_ops) ast_test_validate_cleanup(test, AST_VECTOR_ADD_SORTED(&sv1, ZZZ, strcmp) == 0, rc, cleanup); ast_test_validate_cleanup(test, AST_VECTOR_ADD_SORTED(&sv1, CCC, strcmp) == 0, rc, cleanup); ast_test_validate_cleanup(test, AST_VECTOR_ADD_SORTED(&sv1, AAA, strcmp) == 0, rc, cleanup); - ast_test_validate_cleanup(test, AST_VECTOR_ADD_SORTED(&sv1, CCC2, strcmp) == 0, rc, cleanup); + ast_test_validate_cleanup(test, AST_VECTOR_ADD_SORTED(&sv1, (char*)CCC2, strcmp) == 0, rc, cleanup); ast_test_validate_cleanup(test, AST_VECTOR_GET(&sv1, 0) == AAA, rc, cleanup); ast_test_validate_cleanup(test, AST_VECTOR_GET(&sv1, 1) == BBB, rc, cleanup); |