AcademySoftwareFoundation / AcademySoftwareFoundation/OpenColorIO
Filerules : non-reentry issue in Config::getRuleFromFilepath public function
まだ誰も着手していません。
- 主要言語
- C++
- スター
- 2.1k
- フォーク
- 505
- PR マージ指標
- 30日以内にマージされた PR はありません
説明
Hello,
I've recently migrated my code from the deprecated Config::parseColorSpaceFromString to the recommended Config::getColorSpaceFromFilepath, and encountered a thread-safety issue.
In FileRules::Impl::getRuleFromFilepath(), the method matches() is called in what appears to be a read-only context, implying it should be thread-safe.
However, when a rule is of type FILE_RULE_PARSE_FILEPATH, the matches() method may modify the internal m_colorspace member as a side effect. This introduces shared mutable state, which breaks thread safety when getColorSpaceFromFilepath() is used concurrently on a shared ConstConfigRcPtr.
This behavior leads to data races or inconsistent results in my multi-threaded application.
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
調査の方向性
src/OpenColorIO/FileRules.cpp の 638 行目付近から始め、FileRules::Impl::getRuleFromFilepath()、matches()、および m_colorspace メンバーを追跡します。共有された ConstConfigRcPtr に対する Config::getColorSpaceFromFilepath() の並行呼び出しを再現し、FILE_RULE_PARSE_FILEPATH パスを調べます。読み取り専用の検索によって共有可変状態、データ競合、または一貫性のない結果が発生しなくなれば完了です。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- cpp
- 領域
- backend
- issue の種類
- バグ
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 活発さ
- 停滞
- 明瞭さ
- おおむね明確
- 初心者へのやさしさ
- 35/100