[sr-dev] [kamailio/kamailio] presence_reginfo: Add option for aggregating reginfo presentities (PR #3240)
Matteo
notifications at github.com
Thu Sep 22 09:35:52 CEST 2022
@xadhoom commented on this pull request.
> + LM_ERR("body string too short!\n");
+ return NULL;
+ }
+ version_start = strstr(body->s + 30, "version=");
+ if(!version_start) {
+ LM_ERR("version string not found!\n");
+ return NULL;
+ }
+ version_start += 9;
+
+ /* safety check for placeholder - if it is body not set by the module,
+ * don't update the version */
+ if(strncmp(version_start, "00000000000\"", 12) != 0)
+ return NULL;
+
+ version_len = snprintf(version, MAX_INT_LEN + 2, "%d\"", subs->version);
Well, does not change the flow but yes, is useful for debugging, added.
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3240#discussion_r977301700
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/3240/review/1116529583 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.kamailio.org/pipermail/sr-dev/attachments/20220922/9dc2227d/attachment-0001.htm>
More information about the sr-dev
mailing list