[vlc-devel] [PATCH] Resolve XML special character on playlist filename

Edward Wang edward.c.wang at compdigitec.com
Sun Sep 29 14:44:22 CEST 2013


On 2013-09-29 04:58, Reka Inovan <reka.inovan at gmail.com> wrote:
> It aims to fix bug #9311, which also affect all other xml based 
> playlist format e.g asx. One that i don't realize is that it also will 
> have that effect on  non-xml based playlist files. But one quick look 
> to resolve_xml_special_chars suggest that it will ignore all ampersand 
> that isn't followed by any known xml entity, so i think they shouldn't 
> affect those non-xml format..
This patch is broken, as it affects M3U files as well.

For example, I have a m3u8 playlist with a file named 'A " B.mp3' 
(literally). With your patch VLC tries to open 'A " B.mp3', which is the 
incorrect behaviour. M3U8 is not XML and should not be affected.

Regards,
         Edward Wang



More information about the vlc-devel mailing list