miconda commented on this pull request.
@@ -595,6 +598,17 @@ static int print_contact_str(char
**dest, str *uri, qvalue_t q, str *path, char
memcpy(p, buf.s, buf.len);
p += buf.len;
}
+
+ /* branch flags (not SIP standard conformant) */
+ buf.s = int2str(flags, &buf.len);
+ if (p + FLAGS_PARAM_LEN + buf.len > end) {
+ return -1;
+ }
+ memcpy(p, FLAGS_PARAM, FLAGS_PARAM_LEN);
+ p += FLAGS_PARAM_LEN;
+ memcpy(p, buf.s, buf.len);
+ p += buf.len;
Seems that follow up patches add the option. For the future, when doing the PR merge the
patches done to the same component in one, makes the review simpler as one does it in the
order they are listed and later discovers is no longer same code, being again changed.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/1243#discussion_r140628661