Module: kamailio Branch: master Commit: f5b63dc85499ffaebffcab15f360d9082edbb065 URL: https://github.com/kamailio/kamailio/commit/f5b63dc85499ffaebffcab15f360d9082edbb065
Author: Henning Westerholt <[email protected]> Committer: Henning Westerholt <[email protected]> Date: 2025-11-05T14:01:08Z db_redis: restore C89 compatiblity by moving variable definition outside for loop --- Modified: src/modules/db_redis/redis_sentinels.c --- Diff: https://github.com/kamailio/kamailio/commit/f5b63dc85499ffaebffcab15f360d9082edbb065.diff Patch: https://github.com/kamailio/kamailio/commit/f5b63dc85499ffaebffcab15f360d9082edbb065.patch --- diff --git a/src/modules/db_redis/redis_sentinels.c b/src/modules/db_redis/redis_sentinels.c index cafdbff0cb1..1dd1368bd72 100644 --- a/src/modules/db_redis/redis_sentinels.c +++ b/src/modules/db_redis/redis_sentinels.c @@ -41,12 +41,14 @@ static int str_eq_reply(const redisReply *r, const char *s) static redisReply *get_field(redisReply *r, const char *field) { + size_t i; + if(!r) return NULL; if(r->type == REDIS_REPLY_ARRAY) { // Treat as alternating field/value list - for(size_t i = 0; i + 1 < r->elements; i += 2) { + for(i = 0; i + 1 < r->elements; i += 2) { redisReply *k = r->element[i]; redisReply *v = r->element[i + 1]; if(str_eq_reply(k, field)) @@ -60,6 +62,8 @@ static redisReply *get_field(redisReply *r, const char *field) static int replica_list_init(redisReply *reply, struct reply_list *list) { + size_t i; + if(!reply || reply->type != REDIS_REPLY_ARRAY || reply->elements == 0) { LM_ERR("Invalid reply for replica list\n"); return -1; @@ -74,7 +78,7 @@ static int replica_list_init(redisReply *reply, struct reply_list *list) list->head = NULL; list->tail = NULL; - for(size_t i = 0; i < reply->elements; i++) { + for(i = 0; i < reply->elements; i++) { struct reply_node *node; /* _______________________________________________ Kamailio - Development Mailing List -- [email protected] To unsubscribe send an email to [email protected] Important: keep the mailing list in the recipients, do not reply only to the sender!
