mirrors/cdt
1
0
Fork 0
mirror of https://github.com/eclipse-cdt/cdt synced 2025-04-21 21:52:10 +02:00
Code Issues Wiki Activity

Bug 569911: Stop allowing OOB records to accumulate forever

Browse source 
Add a limit of 100,000 records to prevent OOM errors. Normally
only a few OOB records are needed, but in some cases like with
tdump, hundreds may be needed.

Change-Id: I967e0facc1dc326d94fa67b1d647417ee3cd8891
This commit is contained in:
Jonah Graham 2020-12-23 20:30:14 -05:00
parent 5f8fa75f1f
commit ee55c6804d
1 changed files with 3 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
dsf-gdb/org.eclipse.cdt.dsf.gdb/src/org/eclipse/cdt/dsf/mi/service/command/AbstractMIControl.java
View file

@ -1088,8 +1088,9 @@ public abstract class AbstractMIControl extends AbstractDsfService implements IM
fAccumulatedOOBRecords.add(oob);
// limit growth, but only if these are not responses to CLI commands
// Bug 302927 & 330608
if (fRxCommands.isEmpty() && fAccumulatedOOBRecords.size() > 20) {
// Bug 302927 & 330608 & 569911
if (fRxCommands.isEmpty() && fAccumulatedOOBRecords.size() > 20
|| fAccumulatedOOBRecords.size() > 100000) {
fAccumulatedOOBRecords.remove(0);
}