@wkampich commented on this pull request.


> +                                             if(oldurl.s != NULL && 
> oldurl.len > 0) {
+                                                       if(str_strcasecmp(&url, 
&oldurl) == 0) {
+                                                               LM_ERR("loop 
detected: "
+                                                                          
"[%.*s]<-->[%.*s]\n",
+                                                                               
oldurl.len, oldurl.s, url.len, url.s);
+                                                               goto err;
+                                                       }
+                                               }
+                                               /* remember the redirect target 
*/
+                                               oldurl.s = 
lost_copy_string(url, &oldurl.len);
+                                               /* clean up */
+                                               
lost_free_findServiceResponse(&fsrdata);
+                                               lost_free_string(&ret);
+                                               /* copy url */
+                                               len = 0;
+                                               urlrep = lost_copy_string(url, 
&len);

curl requires a zero terminated string as url, which is not added by the 
http_client API query function `http_client_query_c`

-- 
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/2675#discussion_r605170809
_______________________________________________
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev

Reply via email to