mirrors/cdt
1
0
Fork 0
mirror of https://github.com/eclipse-cdt/cdt synced 2025-06-08 10:16:03 +02:00
Code Issues Wiki Activity

Fixed NPE's resulting from CDebugTarget now nulling out its CDI reference on cleanup.

Browse source
This commit is contained in:
John Cortell 2008-07-29 21:31:11 +00:00
parent 18540abf27
commit d8aad9f2f0
2 changed files with 7 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
debug/org.eclipse.cdt.debug.core/src/org/eclipse/cdt/debug/internal/core/model/CDebugElement.java
View file

@ -207,7 +207,8 @@ abstract public class CDebugElement extends PlatformObject implements ICDebugEle
* @return the CDI session
*/
public ICDISession getCDISession() {
return getCDITarget().getSession();
final ICDITarget cdiTarget = getCDITarget();
return cdiTarget != null ? cdiTarget.getSession() : null;
}
/**

6
debug/org.eclipse.cdt.debug.core/src/org/eclipse/cdt/debug/internal/core/model/CMemoryBlockExtension.java
View file

@ -15,6 +15,7 @@ import java.util.Arrays;
import java.util.HashSet;
import org.eclipse.cdt.debug.core.CDebugCorePlugin;
import org.eclipse.cdt.debug.core.cdi.CDIException;
import org.eclipse.cdt.debug.core.cdi.ICDISession;
import org.eclipse.cdt.debug.core.cdi.event.ICDIEvent;
import org.eclipse.cdt.debug.core.cdi.event.ICDIEventListener;
import org.eclipse.cdt.debug.core.cdi.event.ICDIMemoryChangedEvent;
@ -472,7 +473,10 @@ public class CMemoryBlockExtension extends CDebugElement implements IMemoryBlock
}
fCDIBlock = null;
}
getCDISession().getEventManager().removeEventListener( this );
final ICDISession cdiSession = getCDISession();
if (cdiSession != null) {
cdiSession.getEventManager().removeEventListener( this );
}
}
/* (non-Javadoc)