342301 - NPE when m2e tries to download sources

IRepositoryRegistry is updated by maven project and settings change
events. Other threads may get null repository instances while registry
is being updated as a response to one of the callback events. Added null
check to protect from NPE should this happen.

Signed-off-by: Igor Fedorenko <igor@ifedorenko.com>

1 files changed, 6 insertions, 2 deletions

diff --git a/org.eclipse.m2e.core/src/org/eclipse/m2e/core/internal/index/NexusIndexManager.java b/org.eclipse.m2e.core/src/org/eclipse/m2e/core/internal/index/NexusIndexManager.java
index 0839230f..cab8419b 100644
--- a/org.eclipse.m2e.core/src/org/eclipse/m2e/core/internal/index/NexusIndexManager.java
+++ b/org.eclipse.m2e.core/src/org/eclipse/m2e/core/internal/index/NexusIndexManager.java
@@ -793,7 +793,9 @@ public class NexusIndexManager implements IndexManager, IMavenProjectChangedList
 
       for(ArtifactRepositoryRef repositoryRef : repositories) {
         IRepository repository = repositoryRegistry.getRepository(repositoryRef);
-        indexes.add(getIndex(repository));
+        if(repository != null) {
+          indexes.add(getIndex(repository));
+        }
       }
     } else {
       for(IRepository repository : repositoryRegistry.getRepositories(IRepositoryRegistry.SCOPE_SETTINGS)) {
@@ -816,7 +818,9 @@ public class NexusIndexManager implements IndexManager, IMavenProjectChangedList
 
       for(ArtifactRepositoryRef repositoryRef : repositories) {
         IRepository repository = repositoryRegistry.getRepository(repositoryRef);
-        indexes.add(getIndex(repository));
+        if(repository != null) {
+          indexes.add(getIndex(repository));
+        }
       }
     }