Bug 295828 - [source lookup] NPE in ContainerSourceContainer with (invalid) linked Resources

1 files changed, 16 insertions, 13 deletions

diff --git a/org.eclipse.debug.core/core/org/eclipse/debug/core/sourcelookup/containers/ContainerSourceContainer.java b/org.eclipse.debug.core/core/org/eclipse/debug/core/sourcelookup/containers/ContainerSourceContainer.java
index 82a0290a0..00c5ff7f4 100644
--- a/org.eclipse.debug.core/core/org/eclipse/debug/core/sourcelookup/containers/ContainerSourceContainer.java
+++ b/org.eclipse.debug.core/core/org/eclipse/debug/core/sourcelookup/containers/ContainerSourceContainer.java
@@ -100,19 +100,22 @@ public abstract class ContainerSourceContainer extends CompositeSourceContainer
 				if (fRootURI == null) {
 					return EMPTY;
 				}
-				// See bug 98090 - we need to handle relative path names
-				IFileStore target = fRootFile.getFileStore(new Path(name));
-				if (target.fetchInfo().exists()) {
-					// We no longer have to account for bug 95832, and URIs take care
-					// of canonical paths (fix to bug 95679 was removed).
-					IFile[] files = fRoot.findFilesForLocationURI(target.toURI());
-					if (isFindDuplicates() && files.length > 1) {
-						for (int i = 0; i < files.length; i++) {
-							sources.add(files[i]);
-						}
-					} else if (files.length > 0) {
-						sources.add(files[0]);
-					}					
+				// bug 295828 root file may be null for an invalid linked resource
+				if (fRootFile != null) {
+	                // See bug 98090 - we need to handle relative path names
+    				IFileStore target = fRootFile.getFileStore(new Path(name));
+    				if (target.fetchInfo().exists()) {
+    					// We no longer have to account for bug 95832, and URIs take care
+    					// of canonical paths (fix to bug 95679 was removed).
+    					IFile[] files = fRoot.findFilesForLocationURI(target.toURI());
+    					if (isFindDuplicates() && files.length > 1) {
+    						for (int i = 0; i < files.length; i++) {
+    							sources.add(files[i]);
+    						}
+    					} else if (files.length > 0) {
+    						sources.add(files[0]);
+    					}					
+    				}
 				}
 			}
 		}