[dvblast-devel] More patches

[JULIAN GARDNER]

You want me to put a few checks in before?





>________________________________
> From: Georgi Chorbadzhiyski <gf at unixsol.org>
>To: JULIAN GARDNER <joolzg at btinternet.com>; Mailing list for DVBlast developers <dvblast-devel at videolan.org> 
>Sent: Thursday, 24 October 2013, 15:17
>Subject: Re: [dvblast-devel] More  patches
> 
>
>Around 10/24/2013 04:10 PM, JULIAN GARDNER scribbled:
>> diff --git a/util.c b/util.c
>> index 949c28e..d29d779 100644
>> --- a/util.c
>> +++ b/util.c
>> @@ -432,7 +432,9 @@ uint8_t **psi_unpack_sections( uint8_t *p_flat_sections, unsigned int i_size ) {
>>          /* Must use allocated section not p_flat_section + offset directly! */
>>          uint8_t *p_section_local = psi_private_allocate();
>>          memcpy( p_section_local, p_section, i_section_len );
>> -        psi_table_section( pp_sections, p_section_local );
>> +        if( !psi_table_section( pp_sections, p_section_local ) {
>> +            // What do we do with this failure
>> +        }
>>  
>>          i_offset += i_section_len;
>>          if ( i_offset >= i_size - 1 )
>
>There are bigger problems here if psi_table_allocate() /just a few lines before/
>fails.
>
>-- 
>Georgi Chorbadzhiyski | http://georgi.unixsol.org/ | http://github.com/gfto/
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/dvblast-devel/attachments/20131024/1035fbec/attachment.html>