From 305d860a36e02fdadae0fcf5149db8d4e0d3e1e0 Mon Sep 17 00:00:00 2001
From: Donat Zenichev <dzenichev@sipwise.com>
Date: Fri, 25 Apr 2025 15:40:08 +0200
Subject: [PATCH] MT#59962 DRedisConnection: no reply means something went
 wrong

Add a default case for `handle_redis_reply()`.
Fixes:

    Dereference after null check (FORWARD_NULL)
    4. var_deref_op: Dereferencing null pointer reply.

Change-Id: I22edbc3cb14f7e8662efe4c607e1502fbb8c56b6
---
 apps/dsm/mods/mod_redis/DRedisConnection.cpp | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/apps/dsm/mods/mod_redis/DRedisConnection.cpp b/apps/dsm/mods/mod_redis/DRedisConnection.cpp
index 7f5a5689..defc0f3d 100644
--- a/apps/dsm/mods/mod_redis/DRedisConnection.cpp
+++ b/apps/dsm/mods/mod_redis/DRedisConnection.cpp
@@ -99,7 +99,11 @@ int DRedisConnection::handle_redis_reply(redisReply *reply, const char* _cmd) {
       ERROR("REDIS Protocol error detected\n");
       disconnect();
       return DB_E_CONNECTION;
-    }    
+
+    default:
+      disconnect();
+      return DB_E_CONNECTION;
+    }
   }
 
   switch (reply->type) {