commit 19d3f6a49b25f44934b58f2eeb3dd89bc1b241cc
author Jean-Baptiste Onofré <jbonofre@apache.org> Fri Jul 10 09:36:13 2020 +0200
committer GitHub <noreply@github.com> Fri Jul 10 09:36:13 2020 +0200
tree a51f07fea0c005d99ee0dca99b5a022982f069b4
parent e01be4f06e419cb9bc48219c2b37a92baa0cf1f8
parent b53bf6aaf1df36aa6bf7598aa065502d8765adbf

Merge pull request #80 from jbonofre/KARAF-6175

[KARAF-6175] Avoid NPE when pushing features repository to the cluster

features/src/main/java/org/apache/karaf/cellar/features/FeaturesSynchronizer.java

diff --git a/features/src/main/java/org/apache/karaf/cellar/features/FeaturesSynchronizer.java b/features/src/main/java/org/apache/karaf/cellar/features/FeaturesSynchronizer.java
index daf8961..5ee5911 100644
--- a/features/src/main/java/org/apache/karaf/cellar/features/FeaturesSynchronizer.java
+++ b/features/src/main/java/org/apache/karaf/cellar/features/FeaturesSynchronizer.java
@@ -247,7 +247,10 @@
                 if (repositoryList != null && repositoryList.length > 0) {
                     for (Repository repository : repositoryList) {
                         try {
-                            if (!clusterRepositories.containsKey(repository.getURI().toString())) {
+                            if (repository != null
+                                    && repository.getURI() != null
+                                    && repository.getName() != null
+                                    && !clusterRepositories.containsKey(repository.getURI().toString())) {
                                 LOGGER.debug("CELLAR FEATURE: pushing repository {} in cluster group {}", repository.getName(), groupName);
                                 // updating cluster state
                                 clusterRepositories.put(repository.getURI().toString(), repository.getName());