Great, thanks a lot! I will close the bug now... Vaclav
On Wed, May 16, 2007 at 01:03:59PM +0200, tzieleniewski wrote:
Ive checked out from CVS, tested without optional parameter and it works fine:)
Thanks Cheers Tomasz
Hi, thanks.
I have corrected the code in CVS. Please could you try it from there? I did the 'next_hop' parameter of msilo functions optional - thus instead of empty string you should omit the parameter [like m_dump() instead of m_dump("")]. Thanks a lot.
with regards, Vaclav
On Wed, May 16, 2007 at 11:56:49AM +0200, tzieleniewski wrote:
Hi,
Sure here they are: #modparam("msilo","registrar","sip:registrar@voip.touk.pl") modparam("msilo", "db_table", "silo") modparam("msilo", "expire_time", 259200) # 72h - 3 days modparam("msilo", "check_time", 30) modparam("msilo", "clean_period", 5) modparam("msilo", "use_contact", 0)
Cheers
Tomasz, it is strange, but it works for me without any patches. Please, could you send me your msilo module parameters?
thanks
VaclavOn Wed, May 16, 2007 at 10:28:29AM +0200, tzieleniewski wrote:
Thanks michal,
I did it and it worked after patch there appeared an analogous error in the m_dump(..) function in the msilo.c I made the similar correction and now everything works fine. I attached my patch for both things:)
Cheers tomasz
Hi, try run patch with one more parameter -l (or --ignore-whitespace). I will attach the patch instead putting it inline next time.
Michal
On St, 2007-05-16 at 09:56 +0200, tzieleniewski wrote: > Hi michal, > > I run the following patch but there was an error, can you please take a look. > my output below: > > ser@sen:~/src/ser-2.0/modules/msilo$ patch -p0 msilo.c < msilo.c.patch > patching file msilo.c > Hunk #1 FAILED at 325. > 1 out of 1 hunk FAILED -- saving rejects to file msilo.c.rej > ser@sen:~/src/ser-2.0/modules/msilo$ cat msilo.c.patch > --- msilo.c Mon May 14 15:28:46 2007 +0200 > +++ msilo.c Mon May 14 17:22:27 2007 +0200 > @@ -325,9 +325,9 @@ > goto error; > } > mode = str1[0] - '0'; > - if (str2) { > + next_hop.len = strlen(str2); > + if (next_hop.len) { > next_hop.s = str2; > - next_hop.len = strlen(str2); > } > > if (get_to_uid(&uid, msg) < 0) { > > Cheers tomasz > > > You can try apply patch like this: > > diff -r f576f4aff98e modules/msilo/msilo.c > > --- a/modules/msilo/msilo.c Mon May 14 15:28:46 2007 +0200 > > +++ b/modules/msilo/msilo.c Mon May 14 17:22:27 2007 +0200 > > @@ -325,9 +325,9 @@ static int m_store(struct sip_msg* msg, > > goto error; > > } > > mode = str1[0] - '0'; > > - if (str2) { > > + next_hop.len = strlen(str2); > > + if (next_hop.len) { > > next_hop.s = str2; > > - next_hop.len = strlen(str2); > > } > > > > if (get_to_uid(&uid, msg) < 0) { > > > > And please fill bug report at tracker. > > Michal > > > > On Po, 2007-05-14 at 15:35 +0200, tzieleniewski wrote: > > > Hi, > > > > > > When I try to store message in the database I get the following errors in the syslog (There are no messages stored in the database): > > > > > > May 14 15:26:40 sen ser[5853]: ERROR: parse_uri: uri too short: <> (0) > > > May 14 15:26:40 sen ser[5853]: ERROR: uri2dst: bad_uri: > > > May 14 15:26:40 sen ser[5853]: t_uac: no socket found > > > > > > And here is ser.cfg: > > > xlog("L_INFO", " route[INBOUND]: (m_store (0, )) \n"); > > > if (m_store ("0", "")) > > > { > > > xlog("L_INFO", " route[INBOUND]: offline message stored \n"); > > > xlog("L_INFO", " route[INBOUND]: (!t_reply(202,Accepted) \n"); > > > if (!t_reply("202", "Accepted")) > > > { > > > xlog("L_INFO", " route[INBOUND]: sl_reply_error() \n"); > > > sl_reply_error(); > > > } > > > } > > > > > > the error appears in the following place accordin to my logging: > > > May 14 15:26:40 sen ser[5853]: route[INBOUND]: (m_store (0, )) > > > May 14 15:26:40 sen ser[5853]: ERROR: parse_uri: uri too short: <> (0) > > > May 14 15:26:40 sen ser[5853]: ERROR: uri2dst: bad_uri: > > > May 14 15:26:40 sen ser[5853]: t_uac: no socket found > > > May 14 15:26:40 sen ser[5853]: route[INBOUND]: offline message stored > > > May 14 15:26:40 sen ser[5853]: route[INBOUND]: (!t_reply(202,Accepted) > > > May 14 15:26:40 sen ser[5853]: route[INBOUND]: drop <MESSAGE> sip:hulk@voip.rd.touk.pl > > > > > > Below is the Message transaction sent through SER: > > > > > > U 2007/05/14 15:26:40.746835 192.168.0.165:7089 -> 192.168.0.165:5060 > > > MESSAGE sip:hulk@voip.rd.touk.pl SIP/2.0. > > > CSeq: 64 MESSAGE. > > > Via: SIP/2.0/UDP 192.168.0.165:7089;branch=z9hG4bK24d4276b-8c00-dc11-982b-00148539376e;rport. > > > From: "Sen" sip:sen@voip.rd.touk.pl;tag=a0d0276b-8c00-dc11-982b-00148539376e. > > > Call-ID: 32c6276b-8c00-dc11-982b-00148539376e@sen. > > > To: "hulk@" sip:hulk@voip.rd.touk.pl. > > > Allow: INVITE,ACK,OPTIONS,BYE,CANCEL,NOTIFY,REFER,MESSAGE. > > > Content-Type: text/plain;charset=UTF-8. > > > Content-Length: 12. > > > Max-Forwards: 70. > > > . > > > test message > > > # > > > U 2007/05/14 15:26:40.790793 192.168.0.165:5060 -> 192.168.0.165:7089 > > > SIP/2.0 202 Accepted. > > > CSeq: 64 MESSAGE. > > > Via: SIP/2.0/UDP 192.168.0.165:7089;branch=z9hG4bK24d4276b-8c00-dc11-982b-00148539376e;rport=7089. > > > From: "Sen" sip:sen@voip.rd.touk.pl;tag=a0d0276b-8c00-dc11-982b-00148539376e. > > > Call-ID: 32c6276b-8c00-dc11-982b-00148539376e@sen. > > > To: "hulk@" sip:hulk@voip.rd.touk.pl;tag=a6a1c5f60faecf035a1ae5b6e96e979a-fd05. > > > Server: Sip EXpress router (2.0.0-rc1 (x86_64/linux)). > > > Content-Length: 0. > > > Warning: 392 192.168.0.165:5060 "Noisy feedback tells: pid=5853 req_src_ip=192.168.0.165 req_src_port=7089 in_uri=sip:hulk@voip.rd.touk.pl out_uri=sip:hulk@voip.rd.touk.pl via_cnt==1". > > > . > > > > > > Thanks for any feedback! > > > Cheers > > > Tomasz > > > > > > > > > _______________________________________________ > > > Serusers mailing list > > > Serusers@lists.iptel.org > > > http://lists.iptel.org/mailman/listinfo/serusers > >
Serusers mailing list Serusers@lists.iptel.org http://lists.iptel.org/mailman/listinfo/serusers