diff options
| author | Mark Salyzyn <salyzyn@google.com> | 2016-09-13 07:28:21 -0700 |
|---|---|---|
| committer | Mark Salyzyn <salyzyn@google.com> | 2016-09-26 08:21:13 -0700 |
| commit | ba1a798fd8073b6069165cc03289fb9399f0495b (patch) | |
| tree | 34bcc9ed14e28afe4b147592b652b434bf59dc97 /libcutils/sockets_unix.cpp | |
| parent | aa00f5852f3c495184a14d1cd8bf39cc5c9be8db (diff) | |
liblog: free event tag map in __android_log_close()
There is no leak since a reference always remained and could get
reused. It just makes sense to also close the event tag map as well
if logging is closed. If we close we also have to fix a tagArray
leak in android_closeEventTagMap().
NB: __android_log_close() already makes an assumption that another
thread is not logging at the same time, which is true in all callers
at this time. There are some partial mitigation strategies if
__android_log_close() is called asynchronously, but not completely
solved at the cost of a lock for every logging call.
Test: gTest liblog-unit-tests
Bug: 30963384
Bug: 31456426
Change-Id: Ib76ad9302dba4d3f35250213f4dfef22498af238
Diffstat (limited to 'libcutils/sockets_unix.cpp')
0 files changed, 0 insertions, 0 deletions