mirrors/cdt
1
0
Fork 0
mirror of https://github.com/eclipse-cdt/cdt synced 2025-04-29 19:45:01 +02:00
Code Issues Wiki Activity

Bug 306555: It is valid for the platform to request more children that what we have

Browse source
This commit is contained in:
Marc Khouzam 2010-05-11 17:54:50 +00:00
parent efccb72208
commit 62f7695977
1 changed files with 8 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
dsf-gdb/org.eclipse.cdt.dsf.gdb/src/org/eclipse/cdt/dsf/mi/service/MIExpressions.java
View file

@ -869,16 +869,20 @@ public class MIExpressions extends AbstractDsfService implements IExpressions, I
protected void handleSuccess() { protected void handleSuccess() {
IExpressionDMContext[] subExpressions = getData(); IExpressionDMContext[] subExpressions = getData();
if (startIndex >= subExpressions.length || startIndex + length > subExpressions.length) { if (startIndex >= subExpressions.length) {
rm.setStatus(new Status(IStatus.ERROR, GdbPlugin.PLUGIN_ID, REQUEST_FAILED, "Invalid range for evaluating sub expressions.", null)); //$NON-NLS-1$ rm.setStatus(new Status(IStatus.ERROR, GdbPlugin.PLUGIN_ID, REQUEST_FAILED, "Invalid range for evaluating sub expressions.", null)); //$NON-NLS-1$
rm.done(); rm.done();
return; return;
} }
IExpressionDMContext[] subRange = new IExpressionDMContext[length]; int realLength = length;
for (int i=0; i<subRange.length; i++) { if (startIndex + length > subExpressions.length) {
subRange[i] = subExpressions[i+startIndex]; realLength = subExpressions.length - startIndex;
} }
IExpressionDMContext[] subRange = new IExpressionDMContext[realLength];
System.arraycopy(subExpressions, startIndex, subRange, 0, realLength);
rm.setData(subRange); rm.setData(subRange);
rm.done(); rm.done();
} }