[vlc-devel] [PATCH 1/2] vlc_fixups: fix crash when NULL is passed to strdup() on OS/2

KO Myung-Hun komh78 at gmail.com
Thu Dec 22 05:26:37 CET 2016


Attach the patch.

KO Myung-Hun wrote:
> Hi/2.
> 
> Francois Cartegnie wrote:
>> Le 20/12/2016 à 13:37, KO Myung-Hun a écrit :
>>> On OS/2, strdup() crashes if NULL is passed.
>>
>> No.
>> Fix the cause, not the expected effect.
>>
> 
> I looked into this further, and found the cause.
> 
> FromCharset() in ExtractIntlStrings() of demux/mp4/meta.c returns NULL.
> At that time, charset is "MACINTOSH//eng" and i_lang is 0.
> 
> MACINTOSH//eng is a correct charset ? libiconv installed on my machine
> does not understand it.
> 

-- 
KO Myung-Hun

Using Mozilla SeaMonkey 2.7.2
Under OS/2 Warp 4 for Korean with FixPak #15
In VirtualBox v4.1.32 on Intel Core i7-3615QM 2.30GHz with 8GB RAM

Korean OS/2 User Community : http://www.ecomstation.co.kr

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-demux-mp4-try-again-with-a-default-charset-if-FromCh.patch
Type: application/x-patch
Size: 1075 bytes
Desc: not available
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20161222/9ea854e6/attachment.bin>


More information about the vlc-devel mailing list