[fix][broker] managedLedger.getConfig().getProperties().putAll(proper…

…ties) NPE (apache#20361) (apache#5932) (cherry picked from commit aa7decc) Co-authored-by: Heesung Sohn <[email protected]> (cherry picked from commit 5f6ed1b)
liangyepianzhou · Oct 24, 2023 · 8765134 · 8765134
1 parent bc7ee1a
commit 8765134
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/impl/PersistentTopicsBase.java b/pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/impl/PersistentTopicsBase.java
@@ -697,7 +697,11 @@ private CompletableFuture<Void> internalUpdateNonPartitionedTopicProperties(Map<
 
                     @Override
                     public void updatePropertiesComplete(Map<String, String> properties, Object ctx) {
+                        if (managedLedger.getConfig().getProperties() == null) {
+                            managedLedger.getConfig().setProperties(new HashMap<>());
+                        }
                         managedLedger.getConfig().getProperties().putAll(properties);
+
                         future.complete(null);
                     }