29 Aug
2013
29 Aug
'13
9:36 a.m.
I thought I replied to this, but actually I couldn't find it, maybe I
just had the intention to do it and then got distristracted by something
else -- thanks for the reminder.
The proposed changes are fine with me, they can be committed.
Daniel
On 8/29/13 2:53 PM, Juha Heinanen wrote:
Daniel-Constantin Mierla writes:
--
Daniel-Constantin Mierla - http://www.asipto.com
http://twitter.com/#!/miconda - http://www.linkedin.com/in/miconda
Kamailio Advanced Trainings - Berlin, Oct 21-24; Miami, Nov 11-13, 2013
- more details about Kamailio trainings at http://www.asipto.com -
I will comment over the rest of proposed changes
once I will check the
sources.
i guess you have been too busy to check this:
> in parser/parse_fline.c, i would like to make
the first three L_DBG and
> change the last to L_ERR:
>
> LOG(L_INFO, "ERROR:parse_first_line: bad %s first line\n",
> (fl->type==SIP_REPLY)?"reply(status)":"request");
>
> LOG(L_INFO, "ERROR: at line 0 char %d: \n", offset );
> prn=pkg_malloc( offset );
> if (prn) {
> for (t=0; t<offset; t++)
> if (*(buffer+t)) *(prn+t)=*(buffer+t);
> else *(prn+t)='°';
> LOG(L_INFO, "ERROR: parsed so far: %.*s\n", offset, ZSW(prn) );
> pkg_free( prn );
> };
> error1:
> fl->type=SIP_INVALID;
> LOG(L_INFO, "ERROR:parse_first_line: bad message\n");
> /* skip line */
> nl=eat_line(buffer,len);
> return nl;
one error level message should be enough about the same issue.
the
first two are clearly debug related.
-- juha