[Date Prev][Date Next][Thread Prev][Thread Next]
[Date Index]
[Thread Index]
- Subject: Re: Maybe a bug in lundump.c
- From: 云风 <cloudwu@...>
- Date: Wed, 19 Mar 2014 19:04:05 +0800
2014-03-19 18:47 GMT+08:00 Luiz Henrique de Figueiredo <lhf@tecgraf.puc-rio.br>:
>> In function : void LoadConstants(LoadState* S, Proto* f)
>>
>> If the constant is string, it will be call setsvalue2n .
>>
>> case LUA_TSTRING:
>> setsvalue2n(S->L,o,LoadString(S));
>>
>> and setsvalue2n crashes if LoadString(S) return NULL.
>
> LoadString should never return NULL when called from LoadConstants.
> There is only one case when LoadString returns NULL: for the chunck source,
> when debug info has been stripped.
>
I mean if the input stream is not generate by luac or string.dump,
LoadString may return NULL .
Do more checking for the corrupted data stream would be better .
--
http://blog.codingnow.com
