[sr-dev] [kamailio/kamailio] lost: do not parse locationURI with indention spaces (#2569)

sergey-safarov notifications at github.com
Tue Dec 1 23:39:59 CET 2020


looks as trim to not strip all trailing symbols.
I have tested
```diff
diff --git a/src/modules/lost/functions.c b/src/modules/lost/functions.c
index c94c7883f0..b152b08300 100644
--- a/src/modules/lost/functions.c
+++ b/src/modules/lost/functions.c
@@ -290,6 +290,9 @@ int lost_held_function(struct sip_msg *_m, char *_con, char *_pidf, char *_url,
                                if(xmlStrcmp(cur_node->name,
                                                        (const xmlChar *)"locationUriSet") == 0) {
 
+                                       str test = {"test\n", 5};
+                                       trim(&test);
+                                       LM_WARN("'%s' after\n", test.s);
                                        LM_DBG("*** node '%s' found\n", cur_node->name);
 
                                        /* get the locationUri element */
```
This produce message in log
```
 1(8906) WARNING: lost [functions.c:307]: lost_held_function(): 'test
' after
```
This produce
```

Looks as trailing `\n` not removed.

-- 
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/issues/2569#issuecomment-736865447
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.kamailio.org/pipermail/sr-dev/attachments/20201201/bac8db9c/attachment.htm>


More information about the sr-dev mailing list