diff options
| author | Joshua Colp <jcolp@digium.com> | 2015-11-14 09:02:10 -0400 |
|---|---|---|
| committer | Joshua Colp <jcolp@digium.com> | 2015-11-14 08:06:48 -0500 |
| commit | a1fcf6f7b28caf8e2f9b808449c86515e2ef90c4 (patch) | |
| tree | 4945aed4016d1590d5eadb69cf2b31dbf7611957 /main/hashtab.c | |
| parent | e8881e177046422690d2e87417a93b253469c379 (diff) | |
hashtab: Add NULL check when destroying iterator.
The hashtab API is pretty NULL tolerant which has resulted
in remaining callers not doing much checks themselves.
Unfortunately the function to destroy an iterator does not
do a NULL check and will result in a crash if passed NULL.
This change fixes that.
ASTERISK-25552 #close
Change-Id: Ic1bf8eec3639e5a440f1c941d3ae3893ac6ed619
Diffstat (limited to 'main/hashtab.c')
| -rw-r--r-- | main/hashtab.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/main/hashtab.c b/main/hashtab.c index c08880c69..5d54fb7aa 100644 --- a/main/hashtab.c +++ b/main/hashtab.c @@ -746,6 +746,8 @@ struct ast_hashtab_iter *ast_hashtab_start_write_traversal(struct ast_hashtab *t void ast_hashtab_end_traversal(struct ast_hashtab_iter *it) { + if (!it) + return; if (it->tab->do_locking) ast_rwlock_unlock(&it->tab->lock); free(it); |