mirror of
https://github.com/ClickHouse/ClickHouse.git
synced 2024-12-11 17:02:25 +00:00
StorageStripeLog: avoid UB for rwlock unlock due to unlock from another thread
This commit is contained in:
parent
89a96cf14d
commit
44abb7588a
@ -228,6 +228,11 @@ public:
|
||||
storage.file_checker.save();
|
||||
|
||||
done = true;
|
||||
|
||||
/// unlock should be done from the same thread as lock, and dtor may be
|
||||
/// called from different thread, so it should be done here (at least in
|
||||
/// case of no exceptions occurred)
|
||||
lock.unlock();
|
||||
}
|
||||
|
||||
private:
|
||||
|
5
tests/queries/0_stateless/01797_StripeLog_rwlock_ub.sql
Normal file
5
tests/queries/0_stateless/01797_StripeLog_rwlock_ub.sql
Normal file
@ -0,0 +1,5 @@
|
||||
DROP TABLE IF EXISTS underlying_01797;
|
||||
CREATE TABLE underlying_01797 (key UInt64) Engine=StripeLog();
|
||||
INSERT INTO FUNCTION remote('127.1', currentDatabase(), underlying_01797) SELECT toUInt64(number) FROM system.numbers LIMIT 1;
|
||||
SELECT * FROM underlying_01797 FORMAT Null;
|
||||
DROP TABLE underlying_01797;
|
Loading…
Reference in New Issue
Block a user