From e6830c9ab4e3e7634a0d5425239ebd717c66547c Mon Sep 17 00:00:00 2001
From: Pawel Piech <pawel.piech@windriver.com>
Date: Mon, 24 Mar 2008 16:22:34 +0000
Subject: [PATCH] [219841] Fixed an NPE in BreakpointsMediator related to
 breakpoints that failed to plant.

---
 .../dd/dsf/debug/service/BreakpointsMediator.java    | 12 +++++++-----
 1 file changed, 7 insertions(+), 5 deletions(-)

diff --git a/plugins/org.eclipse.dd.dsf.debug/src/org/eclipse/dd/dsf/debug/service/BreakpointsMediator.java b/plugins/org.eclipse.dd.dsf.debug/src/org/eclipse/dd/dsf/debug/service/BreakpointsMediator.java
index 8b011e0a79f..9b00519853f 100644
--- a/plugins/org.eclipse.dd.dsf.debug/src/org/eclipse/dd/dsf/debug/service/BreakpointsMediator.java
+++ b/plugins/org.eclipse.dd.dsf.debug/src/org/eclipse/dd/dsf/debug/service/BreakpointsMediator.java
@@ -420,13 +420,15 @@ public class BreakpointsMediator extends AbstractDsfService implements IBreakpoi
 				new DataRequestMonitor<IBreakpointDMContext>(getExecutor(), installRM) {
 				@Override
                 protected void handleCompleted() {
-					if (getStatus().isOK()) {
+                    List<IBreakpointDMContext> list = breakpointIDs.get(breakpoint);
+                    if (list == null) {
+                        list = new LinkedList<IBreakpointDMContext>();
+                        breakpointIDs.put(breakpoint, list);
+                    }
+                    
+                    if (getStatus().isOK()) {
 						// Add the breakpoint back-end mapping
-						List<IBreakpointDMContext> list = breakpointIDs.get(breakpoint);
-						if (list == null)
-							list = new LinkedList<IBreakpointDMContext>();
 						list.add(getData());
-						breakpointIDs.put(breakpoint, list);
 					} else {
                         // TODO (bug 219841): need to add breakpoint error status tracking
                         // in addition to fBreakpointDMContexts.