[SR-Users] [NATHELPER] bug in SDP parsing when dealing with multi-part bodies ?

[Pascal Maugeri]

Yes Iñaki, you're right, sorry for this mistake.

this is the existing code:

>         switch(hdr->type){
>                 case HDR_CONTENTTYPE_T:
>                 case HDR_CONTENTDISPOSITION_T:

that I replaced with the following:

>         switch(hdr->type){
>                 case HDR_CONTENTTYPE_T:
>                 case HDR_CONTENTDISPOSITION_T:
>                 case HDR_CONTENTLENGTH_T:

Cheers
Pascal


On Wed, May 12, 2010 at 5:41 PM, Iñaki Baz Castillo <ibc at aliax.net> wrote:

> 2010/5/12 Pascal Maugeri <pascal.maugeri at gmail.com>:
>
> > It looks the Content-length within the multi-part is not accepted.
> > I fixed it changing the source code of parser/sdp/sdp_helpr_funcs.c,
> > in get_sdp_hdr_field() function:
> >         switch(hdr->type){
> >                 case HDR_CONTENTTYPE_T:
> >                 case HDR_CONTENTDISPOSITION_T:
> >                 case HDR_CONTENTLENGTH_T:
> >
> >
> > replaced with:
> >
> >         switch(hdr->type){
> >                 case HDR_CONTENTTYPE_T:
> >                 case HDR_CONTENTDISPOSITION_T:
> >
> > (I'm using kamailio 3.0.1)
>
> Hi, perhaps you mean exactly the opposite (the first is your
> modification and the second the current code)?
> I'm inspecting parser/sdp/sdp_helpr_funcs.c and I don't see "case
> HDR_CONTENTLENGTH_T".
>
> Regards.
>
>
> --
> Iñaki Baz Castillo
> <ibc at aliax.net>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.sip-router.org/pipermail/sr-users/attachments/20100513/1016a2db/attachment.htm>