Merge lp:~townsend/unity/fix-lp926979-trusty into lp:unity/7.2

Proposed by Christopher Townsend
Status: Merged
Approved by: Stephen M. Webb
Approved revision: no longer in the source branch.
Merged at revision: 3826
Proposed branch: lp:~townsend/unity/fix-lp926979-trusty
Merge into: lp:unity/7.2
Diff against target: 14 lines (+3/-1)
1 file modified
dash/DashView.cpp (+3/-1)
To merge this branch: bzr merge lp:~townsend/unity/fix-lp926979-trusty
Reviewer Review Type Date Requested Status
Stephen M. Webb (community) Approve
PS Jenkins bot (community) continuous-integration Approve
Review via email: mp+259288@code.launchpad.net

Commit message

Make sure scope_views_[filter.id] exists before using operator[] on it.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Stephen M. Webb (bregma) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'dash/DashView.cpp'
--- dash/DashView.cpp 2014-07-24 14:03:10 +0000
+++ dash/DashView.cpp 2015-05-15 19:53:47 +0000
@@ -1174,7 +1174,9 @@
11741174
1175 if (!filter.filters.empty())1175 if (!filter.filters.empty())
1176 {1176 {
1177 scope_views_[filter.id]->filters_expanded = true;1177 if (scope_views_.find(filter.id) != std::end(scope_views_))
1178 scope_views_[filter.id]->filters_expanded = true;
1179
1178 // update the scope for each filter1180 // update the scope for each filter
1179 for (auto p : filter.filters) {1181 for (auto p : filter.filters) {
1180 UpdateScopeFilter(filter.id, p.first, p.second);1182 UpdateScopeFilter(filter.id, p.first, p.second);

Subscribers

People subscribed via source and target branches

to status/vote changes: