Commit 2b4053d
crypto: initialize KeyObjectData mutex eagerly
Create the mutex before sharing key data so every copy uses one lock.
Signed-off-by: Filip Skokan <panva.ip@gmail.com>
PR-URL: #64851
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>1 parent 4b7b2ad commit 2b4053d
1 file changed
Lines changed: 4 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -952,10 +952,13 @@ KeyObjectData::KeyObjectData(std::nullptr_t) | |||
| 952 | 952 | ||
| 953 | 953 | KeyObjectData::KeyObjectData(ByteSource symmetric_key) | |
| 954 | 954 | : key_type_(KeyType::kKeyTypeSecret), | |
| 955 | + mutex_(std::make_shared<Mutex>()), | ||
| 955 | 956 | data_(std::make_shared<Data>(std::move(symmetric_key))) {} | |
| 956 | 957 | ||
| 957 | 958 | KeyObjectData::KeyObjectData(KeyType type, EVPKeyPointer&& pkey) | |
| 958 | - : key_type_(type), data_(std::make_shared<Data>(std::move(pkey))) {} | ||
| 959 | + : key_type_(type), | ||
| 960 | + mutex_(std::make_shared<Mutex>()), | ||
| 961 | + data_(std::make_shared<Data>(std::move(pkey))) {} | ||
| 959 | 962 | ||
| 960 | 963 | void KeyObjectData::Data::MemoryInfo(MemoryTracker* tracker) const { | |
| 961 | 964 | if (asymmetric_key) { | |
0 commit comments