[MPlayer-dev-eng] [Patch] libmpdemux/asf_mmst_streaming.c: fixed mms url encoding with multibyte characters

D Richard Felker III dalias at aerifal.cx
Sat Oct 4 20:54:37 CEST 2003


On Sun, Oct 05, 2003 at 01:06:57AM +0800, Wang WenRui wrote:
> Hi,
> 
>   Current mplayer (mine is mplayer-1.0-pre1cvs20031001) cannot play mms
>   stream with multibyte characters in the url.
> 
>   There is a bug in string_utf16() of libmpdemux/asf_mmst_streaming.c:
>   the url encoding code is written for single-byte character only. It
>   just fill a zero byte between every two adjacent bytes of the url
>   string. This is wrong when the url string contains multi-byte
>   characters.
> 
>   My patch is in the attachment, and has been tested working. It uses
>   iconv() to convert the url's encoding to utf-16 correctly. So can this
>   patch be accepted and commit into CVS?

Shouldn't it be UTF8? UTF16 is pointless and just wastes memory and
creates endian-dependence issues...

Rich



More information about the MPlayer-dev-eng mailing list