Module: kamailio
Branch: master
Commit: 4d2988a79d992ed8f3ce4cffa6c387ef0425be94
URL: 
https://github.com/kamailio/kamailio/commit/4d2988a79d992ed8f3ce4cffa6c387ef0425be94

Author: Daniel-Constantin Mierla <[email protected]>
Committer: Daniel-Constantin Mierla <[email protected]>
Date: 2024-07-05T10:41:09+02:00

ndb_redis: print server spec lenght in error message

- use standard pkg alloc error message

---

Modified: src/modules/ndb_redis/redis_client.c

---

Diff:  
https://github.com/kamailio/kamailio/commit/4d2988a79d992ed8f3ce4cffa6c387ef0425be94.diff
Patch: 
https://github.com/kamailio/kamailio/commit/4d2988a79d992ed8f3ce4cffa6c387ef0425be94.patch

---

diff --git a/src/modules/ndb_redis/redis_client.c 
b/src/modules/ndb_redis/redis_client.c
index c9280991ff3..cba94a38c06 100644
--- a/src/modules/ndb_redis/redis_client.c
+++ b/src/modules/ndb_redis/redis_client.c
@@ -944,12 +944,13 @@ int check_cluster_reply(redisReply *reply, 
redisc_server_t **rsrv)
                                                addr.len, addr.s, port);
 
                                if(server_len < 0 || server_len > 
sizeof(spec_new) - 1) {
-                                       LM_ERR("failed to print server spec 
string\n");
+                                       LM_ERR("failed to print server spec 
string (%d)\n",
+                                                       server_len);
                                        return 0;
                                }
                                server_new = (char *)pkg_malloc(server_len + 1);
                                if(server_new == NULL) {
-                                       LM_ERR("Error allocating pkg mem\n");
+                                       PKG_MEM_ERROR;
                                        return 0;
                                }
 
@@ -969,7 +970,7 @@ int check_cluster_reply(redisReply *reply, redisc_server_t 
**rsrv)
                                                                        
name.len, name.s);
                                                        return 1;
                                                } else {
-                                                       LM_ERR("ERROR 
connecting to the new server with "
+                                                       LM_ERR("failed 
connecting to the new server with "
                                                                   "name: 
%.*s\n",
                                                                        
name.len, name.s);
                                                        return 0;

_______________________________________________
Kamailio (SER) - Development Mailing List
To unsubscribe send an email to [email protected]

Reply via email to