@wkampich commented on this pull request.


> +                                                     goto err;
+                                               }
+                                               /* clean up */
+                                               tmp.s = NULL;
+                                               tmp.len = 0;
+                                               /* check loop */
+                                               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);

thanks, have incorporated it

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

Reply via email to