We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
NotificationCenter
Describe the bug All observers container access is protected by mutex, except access by destructor destroying the container.
To Reproduce Thread sanitizer
Expected behavior Clean destruction
Logs
WARNING: ThreadSanitizer: data race (pid=70947) Write of size 8 at 0x0001070011c8 by main thread: // ... #11 Poco::NotificationCenter::~NotificationCenter() NotificationCenter.cpp:31 (libPocoFoundationd.100.dylib:arm64+0x104418) #12 Poco::TaskManager::~TaskManager() TaskManager.cpp:52 (libPocoFoundationd.100.dylib:arm64+0x1cb4a0) // ... Previous read of size 8 at 0x0001070011c8 by thread T11 (mutexes: write M0): // ... #8 Poco::NotificationCenter::postNotification(Poco::AutoPtr<Poco::Notification>) NotificationCenter.cpp:72 (libPocoFoundationd.100.dylib:arm64+0x1050bc) // ..
Please add relevant environment information: osx clang
The text was updated successfully, but these errors were encountered:
fix(NotificationCenter): data race #4307
cbfbf96
fix(NotificationCenter): disable and clear observers in dtor (#4307)
9e8f0ad
4307/8/9/10 data races (#4312)
1e90f64
* fix(NumericString): properly mark uIntToString deprecated #4304 * dev(runLibtests): allow to specify test to run * fix(NotificationCenter): data race #4307 * fix(DirectoryWatcher): data race #4308 * fix(ArchiveStrategy): data race #4309 * fix(ActiveThread): data race #4310 * fix(Task): Cancelled Task shouldn't start running #4311 (WIP) * fix(String): ignore clang loop unrolling warnings * fix(TaskManager): task ownership #4311 * chore(FIFOEventTest): fix unused var warning; disable benchmark in test * fix(Task): remove unnecessary mutex (and prevent cyclic locking reported by TSAN) * fix(CryptoTest): disable testEncryptDecryptGCM * fix(ci): typo * fix(NotificationCenter): disable and clear observers in dtor (#4307) --------- Co-authored-by: Matej Kenda <matejken@gmail.com>
aleks-f
No branches or pull requests
Describe the bug
All observers container access is protected by mutex, except access by destructor destroying the container.
To Reproduce
Thread sanitizer
Expected behavior
Clean destruction
Logs
Please add relevant environment information:
osx clang
The text was updated successfully, but these errors were encountered: