From 8c309586cde08f1c2cd04cd4f8ba14cd84c9d906 Mon Sep 17 00:00:00 2001
From: Alena Laskavaia <elaskavaia.cdt@gmail.com>
Date: Wed, 11 Jun 2008 13:36:40 +0000
Subject: [PATCH] [236214] - fixed potential NPE

---
 .../eclipse/cdt/debug/internal/core/CBreakpointManager.java  | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/debug/org.eclipse.cdt.debug.core/src/org/eclipse/cdt/debug/internal/core/CBreakpointManager.java b/debug/org.eclipse.cdt.debug.core/src/org/eclipse/cdt/debug/internal/core/CBreakpointManager.java
index 1a684bf724a..42ac41a5ebf 100644
--- a/debug/org.eclipse.cdt.debug.core/src/org/eclipse/cdt/debug/internal/core/CBreakpointManager.java
+++ b/debug/org.eclipse.cdt.debug.core/src/org/eclipse/cdt/debug/internal/core/CBreakpointManager.java
@@ -1202,13 +1202,14 @@ public class CBreakpointManager implements IBreakpointsListener, IBreakpointMana
 				// ICDIFunctionBreakpoint on function elements from binary objects can be
 				// set without having a source handle. For this case of line breakpoint 
 				// don't try to match breakpoints with source locator of contained CDebugTarget.
-				String handle = "";
+				String handle = null;
 				try {
 					handle = breakpoint.getSourceHandle();
 				} catch (CoreException ex) {
 					// ignore exception. source handle will be empty anyway.
 				}
-				result = handle.length() > 0;
+				result = (handle != null) && (handle.length() > 0);
+
 			}
 		}
 		return result;