From bba9625b2544c5fc05e33ee6c7b65b25b0366427 Mon Sep 17 00:00:00 2001 From: Lucian Balaceanu Date: Mon, 17 Feb 2025 18:07:04 +0200 Subject: [PATCH] MT#55283 fix calling redis_delete when call is not persisted to db Fixes a corner case that happens when trying to delete a call created with ng offer and no-redis-update flag and for which no ng answer was received. In such cases, one would receive warning messages from Redis "ERR DB index is out of range". What happens is that on call creation redis_hosted_db defaults to -1. This value is changed when writing to Redis, but the writing is not done for a ng offer with no-redis-update. Kudos go to Pawel Kuzak. Closes #1910 Change-Id: Iad19ec8ac69b169e6352662697b17eb23e6970fb --- daemon/call.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/daemon/call.c b/daemon/call.c index fb188563f..c1bd3f4d4 100644 --- a/daemon/call.c +++ b/daemon/call.c @@ -4005,7 +4005,8 @@ void call_destroy(call_t *c) { statistics_update_ip46_inc_dec(c, CMC_DECREMENT); statistics_update_foreignown_dec(c); - redis_delete(c, rtpe_redis_write); + if (c->redis_hosted_db >= 0) + redis_delete(c, rtpe_redis_write); __call_iterator_remove(c);