mia/Aegisub
1
0
Fork 1
Code Issues Pull requests Projects Releases Wiki Activity

Fix a use-after-free when a SubsEditCtrl is destroyed while the thesaurus is loading

Browse source
This commit is contained in:
Thomas Goyne 2014-07-10 19:57:19 -07:00
parent 4dc38447dc
commit 548fbd814b
2 changed files with 13 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
src/thesaurus.cpp
View file

@ -42,7 +42,7 @@ Thesaurus::Thesaurus()
}
Thesaurus::~Thesaurus() {
// Explicit empty destructor needed for scoped_ptr with incomplete types
if (cancel_load) *cancel_load = true;
}
std::vector<Thesaurus::Entry> Thesaurus::Lookup(std::string word) {
@ -98,11 +98,18 @@ void Thesaurus::OnLanguageChanged() {
LOG_I("thesaurus/file") << "Using thesaurus: " << dat;
if (cancel_load) *cancel_load = true;
cancel_load = new bool{false};
auto cancel = cancel_load; // Needed to avoid capturing via `this`
agi::dispatch::Background().Async([=]{
try {
auto thes = agi::make_unique<agi::Thesaurus>(dat, idx);
agi::dispatch::Main().Sync([&thes, this]{
impl = std::move(thes);
agi::dispatch::Main().Sync([&thes, cancel, this]{
if (!*cancel) {
impl = std::move(thes);
cancel_load = nullptr;
}
delete cancel;
});
}
catch (agi::Exception const& e) {

3
src/thesaurus.h
View file

@ -39,6 +39,9 @@ class Thesaurus {
agi::signal::Connection dict_path_listener;
/// Thesaurus path change handler
void OnPathChanged();
bool *cancel_load = nullptr;
public:
/// A pair of a word and synonyms for that word
typedef std::pair<std::string, std::vector<std::string>> Entry;