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

Author: Daniel-Constantin Mierla <mico...@gmail.com>
Committer: Daniel-Constantin Mierla <mico...@gmail.com>
Date: 2019-12-02T16:21:25+01:00

nsq: use snprintf() instead of sprintf()

---

Modified: src/modules/nsq/nsq_reader.c

---

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

---

diff --git a/src/modules/nsq/nsq_reader.c b/src/modules/nsq/nsq_reader.c
index 9a472ca8df..a8f403bda9 100644
--- a/src/modules/nsq/nsq_reader.c
+++ b/src/modules/nsq/nsq_reader.c
@@ -92,23 +92,23 @@ int nsq_consumer_event(char *payload, char *channel, char 
*topic)
        json_api.extract_field(json_obj, k, &ev_name);
        pkg_free(k);
 
-       sprintf(buffer, "nsq:consumer-event-%.*s-%.*s",ev_category.len, 
ev_category.s, ev_name.len, ev_name.s);
+       snprintf(buffer, 512, "nsq:consumer-event-%.*s-%.*s",ev_category.len, 
ev_category.s, ev_name.len, ev_name.s);
        for (p=buffer ; *p; ++p) *p = tolower(*p);
        for (p=buffer ; *p; ++p) if(*p == '_') *p = '-';
        if (nsq_consumer_fire_event(buffer) != 0) {
-               sprintf(buffer, "nsq:consumer-event-%.*s", ev_category.len, 
ev_category.s);
+               snprintf(buffer, 512, "nsq:consumer-event-%.*s", 
ev_category.len, ev_category.s);
                for (p=buffer ; *p; ++p) *p = tolower(*p);
                for (p=buffer ; *p; ++p) if(*p == '_') *p = '-';
                if (nsq_consumer_fire_event(buffer) != 0) {
-                       sprintf(buffer, "nsq:consumer-event-%.*s-%.*s", 
nsq_event_key.len, nsq_event_key.s, nsq_event_sub_key.len, nsq_event_sub_key.s);
+                       snprintf(buffer, 512, "nsq:consumer-event-%.*s-%.*s", 
nsq_event_key.len, nsq_event_key.s, nsq_event_sub_key.len, nsq_event_sub_key.s);
                        for (p=buffer ; *p; ++p) *p = tolower(*p);
                        for (p=buffer ; *p; ++p) if(*p == '_') *p = '-';
                        if (nsq_consumer_fire_event(buffer) != 0) {
-                               sprintf(buffer, "nsq:consumer-event-%.*s", 
nsq_event_key.len, nsq_event_key.s);
+                               snprintf(buffer, 512, 
"nsq:consumer-event-%.*s", nsq_event_key.len, nsq_event_key.s);
                                for (p=buffer ; *p; ++p) *p = tolower(*p);
                                for (p=buffer ; *p; ++p) if(*p == '_') *p = '-';
                                if (nsq_consumer_fire_event(buffer) != 0) {
-                                       sprintf(buffer, "nsq:consumer-event");
+                                       snprintf(buffer, 512, 
"nsq:consumer-event");
                                        if (nsq_consumer_fire_event(buffer) != 
0) {
                                                LM_ERR("nsq:consumer-event not 
found");
                                        }


_______________________________________________
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev

Reply via email to