@ivanuschak commented on this pull request.


> +                     return 1;
+               }
+       }
+}
+
+int check_flow_token(struct sip_msg *msg)
+{
+       struct hdr_field *hdr;
+       struct sip_uri puri;
+       rr_t *rt;
+       str uri;
+       int ret;
+       struct receive_info *rcv = NULL;
+       tcp_connection_t *con = NULL;
+
+       switch(find_first_route(msg)) {

please check `find_first_route` function implementation. It returns 4 values: 
-2, -1, 0, 1.
The value `1` is returned when no route header found.
Service crashes when execution goes over this function and `sip_msg` does not 
contain Route header. Need to add `1` in the switch.
Please fix.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/4449#pullrequestreview-3390558606
You are receiving this because you are subscribed to this thread.

Message ID: <kamailio/kamailio/pull/4449/review/[email protected]>
_______________________________________________
Kamailio - Development Mailing List -- [email protected]
To unsubscribe send an email to [email protected]
Important: keep the mailing list in the recipients, do not reply only to the 
sender!

Reply via email to