hdikme commented on this pull request.


> @@ -1675,6 +1677,20 @@ static int build_rtpp_socks() {
                                LM_WARN("Failed enable set MTU discovery socket 
option\n");
 #endif
 
+                       if((0 <= control_cmd_tos) && (control_cmd_tos < 256)) {
+                               unsigned char tos = control_cmd_tos;
+                               if (pnode->rn_umode == 6) {
+                                       setsockopt(rtpp_socks[pnode->idx], 
IPPROTO_IPV6,
+                                                       IPV6_TCLASS, &tos,
+                                                       sizeof(tos));

I have seen in the following link (https://notes.shichao.io/unp/ch7/) that the 
data type for IP_TOS is "int". Therefore i also changed for IPv4, otherwise 
your comment was clear. Anyway, i have changed it based on your review again, 
thank you for your time.

-- 
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/1442#discussion_r169303694
_______________________________________________
Kamailio (SER) - Development Mailing List
[email protected]
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev

Reply via email to