[vlc-devel] [PATCH] added handling of wrong MIME boundary separator to mjpeg demux

[Sergey Radionov]

вторник, 11 сентября 2012 г. пользователь Sergey Radionov писал:

> ---
>  modules/demux/mjpeg.c |   53
> ++++++++++++++++++++++++-------------------------
>  1 files changed, 26 insertions(+), 27 deletions(-)
>
> diff --git a/modules/demux/mjpeg.c b/modules/demux/mjpeg.c
> index a057db3..b6c7445 100644
> --- a/modules/demux/mjpeg.c
> +++ b/modules/demux/mjpeg.c
> @@ -193,33 +193,8 @@ static bool CheckMimeHeader( demux_t *p_demux, int
> *p_header_size )
>      if( strncmp( (char *)p_sys->p_peek, "--", 2 ) != 0
>          && strncmp( (char *)p_sys->p_peek, "\r\n--", 4 ) != 0 )
>      {
> -        /* Some broken stream may lack the first boundary */
> -        if ( p_sys->psz_separator == NULL )
> -        {
> -            msg_Warn( p_demux, "Malformed stream. Trying to work around");
> -            char *content_type = stream_ContentType( p_demux->s );
> -            if ( content_type == NULL )
> -                return false;
> -            const char* boundary = strstr( content_type, "boundary=--" );
> -            if ( boundary != NULL )
> -            {
> -                p_sys->psz_separator = strdup( boundary + strlen(
> "boundary=--" ) );
> -                msg_Dbg( p_demux, "Video boundary extracted from
> Content-Type: %s", p_sys->psz_separator );
> -                free( content_type );
> -                /* Skip to HTTP header parsing as there's no boundary to
> extract
> -                 * from the stream */
> -            }
> -            else
> -            {
> -                free( content_type );
> -                return false;
> -            }
> -        }
> -        else
> -        {
> -            *p_header_size = 0;
> -            return false;
> -        }
> +        *p_header_size = 0;
> +        return false;
>      }
>      else
>      {
> @@ -351,6 +326,22 @@ static int Open( vlc_object_t * p_this )
>          goto error;
>      }
>
> +    char *content_type = stream_ContentType( p_demux->s );
> +    if ( content_type )
> +    {
> +        //FIXME: this is not fully match to RFC
> +        char* boundary = strstr( content_type, "boundary=" );
> +        if( boundary )
> +        {
> +            p_sys->psz_separator = strdup( boundary + strlen("boundary=")
> );
> +            if( !p_sys->psz_separator ) {
> +                free( content_type );
> +                goto error;
> +            }
> +        }
> +        free( content_type );
> +    }
> +
>      b_matched = CheckMimeHeader( p_demux, &i_size);
>      if( b_matched )
>      {
> @@ -409,6 +400,7 @@ static int Open( vlc_object_t * p_this )
>      return VLC_SUCCESS;
>
>  error:
> +    free( p_sys->psz_separator );
>      free( p_sys );
>      return VLC_EGENERIC;
>  }
> @@ -532,6 +524,13 @@ static int MimeDemux( demux_t *p_demux )
>          {
>              break;
>          }
> +        //sometimes in Content-Type the boundary is defined something
> like "--video boundary--"
> +        else if( !strncmp( p_sys->psz_separator, "--", 2 ) &&
> +                 !strncmp( p_sys->psz_separator, (char *)(p_sys->p_peek +
> i),
> +                 strlen( p_sys->psz_separator ) ) )
> +        {
> +            break;
> +        }
>
>          i++;
>          i_size++;
> --
> 1.7.7.1.msysgit.0
>
>
Any comments please?
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20120914/b2ab555b/attachment.html>