Skip to content

Commit 4215fdc

Browse files
authored
Check the correct container during writeback (#236)
1 parent 9714cd6 commit 4215fdc

File tree

1 file changed

+8
-8
lines changed

1 file changed

+8
-8
lines changed

src/model_repository_manager.cc

+8-8
Original file line numberDiff line numberDiff line change
@@ -2156,10 +2156,10 @@ ModelRepositoryManager::DependencyGraph::Writeback(
21562156
for (const auto& model_id : affected_models) {
21572157
// An affected model can be deleted, so the presence of the model must be
21582158
// checked.
2159-
auto node = FindNode(model_id, false /* allow_fuzzy_matching */);
2160-
if (node != nullptr) {
2161-
auto updated_node = updated_dependency_graph.FindNode(
2162-
model_id, false /* allow_fuzzy_matching */);
2159+
auto* updated_node = updated_dependency_graph.FindNode(
2160+
model_id, false /* allow_fuzzy_matching */);
2161+
if (updated_node != nullptr) {
2162+
auto* node = FindNode(model_id, false /* allow_fuzzy_matching */);
21632163
// Writeback
21642164
node->status_ = updated_node->status_;
21652165
node->checked_ = updated_node->checked_;
@@ -2195,10 +2195,10 @@ ModelRepositoryManager::ModelInfoMap::Writeback(
21952195
for (auto& model_id : affected_models) {
21962196
// An affected model can be deleted, so the presence of the model must be
21972197
// checked.
2198-
auto itr = map_.find(model_id);
2199-
if (itr != map_.end()) {
2200-
auto info = itr->second.get();
2201-
auto updated_info = updated_model_info.map_.at(model_id).get();
2198+
auto itr = updated_model_info.map_.find(model_id);
2199+
if (itr != updated_model_info.map_.end()) {
2200+
auto* updated_info = itr->second.get();
2201+
auto* info = map_.at(model_id).get();
22022202
// Writeback
22032203
info->mtime_nsec_ = updated_info->mtime_nsec_;
22042204
}

0 commit comments

Comments
 (0)