diff options
author | Miloslav Trmač <mitr@redhat.com> | 2012-07-18 17:25:30 +0200 |
---|---|---|
committer | Miloslav Trmač <mitr@redhat.com> | 2012-07-18 17:25:30 +0200 |
commit | 32e918922da9cecf4ebd9013fe64e57503ad44cc (patch) | |
tree | 5ff4b779f6d7707330a176f17d8d5c1fce3d1442 /README.rst | |
parent | f4ddd0b5c32b2c3efa85bdd8ca5dde71cbf65be0 (diff) | |
download | libumberlog-32e918922da9cecf4ebd9013fe64e57503ad44cc.tar.gz libumberlog-32e918922da9cecf4ebd9013fe64e57503ad44cc.tar.xz libumberlog-32e918922da9cecf4ebd9013fe64e57503ad44cc.zip |
Let libc manage the syslog mask.
glibc has a per-process, not per-thread, syslog mask, so the per-thread
mask in ul_sys_settings may end up desynchronized from the glibc mask,
breaking compatibility for LD_PRELOAD.
Just call setlogmask (0) to always read the glibc mask, and drop the
per-thread cache.
An alternative is to use a per-thread mask for ul_* calls and to use the
glibc mask for the LD_PRELOAD-overridden functions, but that seems to be
unnecessarily complex.
ul_setlogmask () is now a trivial wrapper, so we don't need to override
setlogmask () at all. We could also drop ul_setlogmask () completely,
but that would break the API.
Signed-off-by: Miloslav Trmač <mitr@redhat.com>
Diffstat (limited to 'README.rst')
0 files changed, 0 insertions, 0 deletions