diff options
| author | Greg Kurz <groug@kaod.org> | 2022-10-21 12:57:34 +0200 |
|---|---|---|
| committer | Paolo Bonzini <pbonzini@redhat.com> | 2022-10-31 09:46:34 +0100 |
| commit | eff3de52f265df39c6fc668415ad317ec1f24051 (patch) | |
| tree | f5cf5a9ca592a20a5361813de78d9bb1a403443c /docs/devel/secure-coding-practices.rst | |
| parent | 19e2a9fb9da067acba95b3be83588bda5a3f6a99 (diff) | |
| download | focaccia-qemu-eff3de52f265df39c6fc668415ad317ec1f24051.tar.gz focaccia-qemu-eff3de52f265df39c6fc668415ad317ec1f24051.zip | |
util/log: Close per-thread log file on thread termination
When `-D ${logfile} -d tid` is passed, qemu_log_trylock() creates
a dedicated log file for the current thread and opens it. The
corresponding file descriptor is cached in a __thread variable.
Nothing is done to close the corresponding file descriptor when the
thread terminates though and the file descriptor is leaked.
The issue was found during code inspection and reproduced manually.
Fix that with an atexit notifier.
Fixes: 4e51069d6793 ("util/log: Support per-thread log files")
Cc: richard.henderson@linaro.org
Signed-off-by: Greg Kurz <groug@kaod.org>
Message-Id: <20221021105734.555797-1-groug@kaod.org>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Diffstat (limited to 'docs/devel/secure-coding-practices.rst')
0 files changed, 0 insertions, 0 deletions