GH-1333: don't delete the expiration index, just avoid the NPE

Signed-off-by: Julien Vermillard <julien@vermillard.com>
eclipse-leshan · Oct 21, 2022 · c96ae82 · c96ae82
1 parent bc07e46
commit c96ae82
Showing 1 changed file with 1 addition and 5 deletions.
diff --git a/...an-server-redis/src/main/java/org/eclipse/leshan/server/redis/RedisRegistrationStore.java b/...an-server-redis/src/main/java/org/eclipse/leshan/server/redis/RedisRegistrationStore.java
@@ -829,11 +829,7 @@ public void run() {
 
                 for (byte[] endpoint : endpointsExpired) {
                     byte[] regBytes = j.get(toEndpointKey(endpoint));
-                    if (regBytes == null) {
-                        LOG.warn("An expiration index points to an unknown registration: '{}'",
-                                new String(endpoint, UTF_8));
-                        j.zrem(EXP_EP, endpoint);
-                    } else {
+                    if (regBytes != null) {
                         Registration r = deserializeReg(regBytes);
                         if (!r.isAlive(gracePeriod)) {
                             Deregistration dereg = removeRegistration(j, r.getId(), true);