From e83d9b7f3575e592cd8b62f64dab33d687fe3223 Mon Sep 17 00:00:00 2001 From: Pawel Piech Date: Fri, 15 Feb 2008 20:42:24 +0000 Subject: [PATCH] [202109] Fixed a bug in the update modes cache which caused occasional invalid updates. --- .../dsf/ui/viewmodel/DefaultVMContentProviderStrategy.java | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/plugins/org.eclipse.dd.dsf.ui/src/org/eclipse/dd/dsf/ui/viewmodel/DefaultVMContentProviderStrategy.java b/plugins/org.eclipse.dd.dsf.ui/src/org/eclipse/dd/dsf/ui/viewmodel/DefaultVMContentProviderStrategy.java index fe27ec0e719..b58052fe967 100644 --- a/plugins/org.eclipse.dd.dsf.ui/src/org/eclipse/dd/dsf/ui/viewmodel/DefaultVMContentProviderStrategy.java +++ b/plugins/org.eclipse.dd.dsf.ui/src/org/eclipse/dd/dsf/ui/viewmodel/DefaultVMContentProviderStrategy.java @@ -325,8 +325,11 @@ public class DefaultVMContentProviderStrategy implements IElementContentProvider // See https://bugs.eclipse.org/bugs/show_bug.cgi?id=202109 // Instead of checking getStatus().isOK(), check getData() != null. if (getData() != null) { - for (int i = 0; i < elementsLength; i++) { - update.setChild(getData().get(i), elementsStartIdx + nodeStartIdx + i); + for (int i = 0; i < elementsLength && i < getData().size(); i++) { + Object child = getData().get(i); + if (child != null) { + update.setChild(getData().get(i), elementsStartIdx + nodeStartIdx + i); + } } } elementsMultiRequestMon.requestMonitorDone(this);