diff options
| author |  Ciaran McCreesh <ciaran.mccreesh@googlemail.com>
| 2012-04-25 19:09:56 +0100 |
|---|---|---|
| committer | Ciaran McCreesh <ciaran.mccreesh@googlemail.com>
| 2012-04-25 19:53:50 +0100 |
| commit | 5dd73b4c8a6b00d12d99e1a246bde0497e1825a2 (patch) | |
| tree | f4cc11b37ffded69f6fafe80d6af72f3f7be7cc6 | |
| parent | 7500aa8b8632b280923024c8a37208bbd59ee9fa (diff) | |
| download | paludis-5dd73b4c8a6b00d12d99e1a246bde0497e1825a2.tar.gz paludis-5dd73b4c8a6b00d12d99e1a246bde0497e1825a2.tar.xz | |
Fix segfault
| -rw-r--r-- | paludis/resolver/decider.cc | 27 |
1 files changed, 14 insertions, 13 deletions
diff --git a/paludis/resolver/decider.cc b/paludis/resolver/decider.cc index 970b67049..edaed7012 100644 --- a/paludis/resolver/decider.cc +++ b/paludis/resolver/decider.cc @@ -1260,19 +1260,20 @@ Decider::find_any_score( const std::shared_ptr<const Resolvents> resolvents_unless_block(is_block ? make_null_shared_ptr() : _get_resolvents_for(spec, reason_unless_block).first); std::list<std::shared_ptr<Decision> > could_install_decisions; - for (Resolvents::ConstIterator r(resolvents_unless_block->begin()), r_end(resolvents_unless_block->end()) ; - r != r_end ; ++r) - { - const std::shared_ptr<Resolution> could_install_resolution(_create_resolution_for_resolvent(*r)); - const std::shared_ptr<ConstraintSequence> could_install_constraints(_make_constraints_from_dependency( - our_resolution, dep, reason_unless_block, si_take)); - for (ConstraintSequence::ConstIterator c(could_install_constraints->begin()), c_end(could_install_constraints->end()) ; - c != c_end ; ++c) - could_install_resolution->constraints()->add(*c); - auto could_install_decision(_try_to_find_decision_for(could_install_resolution, false, false, false, false, false)); - if (could_install_decision) - could_install_decisions.push_back(could_install_decision); - } + if (resolvents_unless_block) + for (Resolvents::ConstIterator r(resolvents_unless_block->begin()), r_end(resolvents_unless_block->end()) ; + r != r_end ; ++r) + { + const std::shared_ptr<Resolution> could_install_resolution(_create_resolution_for_resolvent(*r)); + const std::shared_ptr<ConstraintSequence> could_install_constraints(_make_constraints_from_dependency( + our_resolution, dep, reason_unless_block, si_take)); + for (ConstraintSequence::ConstIterator c(could_install_constraints->begin()), c_end(could_install_constraints->end()) ; + c != c_end ; ++c) + could_install_resolution->constraints()->add(*c); + auto could_install_decision(_try_to_find_decision_for(could_install_resolution, false, false, false, false, false)); + if (could_install_decision) + could_install_decisions.push_back(could_install_decision); + } /* next: could install, and something similar is installed */ if (! is_block) |