<div dir="ltr">On Sat, Sep 27, 2014 at 2:57 PM, Rémi Denis-Courmont <span dir="ltr"><<a href="mailto:remi@remlab.net" target="_blank">remi@remlab.net</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Le samedi 27 septembre 2014, 14:29:36 Tristan Matthews a écrit :<br>
<span class="">> This patch handles allocation failures and encoding failure.<br>
><br>
> ---<br>
>  modules/codec/twolame.c | 15 +++++++++++++++<br>
>  1 file changed, 15 insertions(+)<br>
><br>
> diff --git a/modules/codec/twolame.c b/modules/codec/twolame.c<br>
> index 20ef952..3255e9b 100644<br>
> --- a/modules/codec/twolame.c<br>
> +++ b/modules/codec/twolame.c<br>
> @@ -274,6 +274,8 @@ static block_t *Encode( encoder_t *p_enc, block_t<br>
> *p_aout_buf ) return NULL;<br>
><br>
>          p_block = block_Alloc( i_used );<br>
> +        if( !p_block )<br>
> +            return NULL;<br>
>          memcpy( p_block->p_buffer, p_sys->p_out_buffer, i_used );<br>
>          p_block->i_length = CLOCK_FREQ *<br>
>                  (mtime_t)MPEG_FRAME_SIZE /<br>
> (mtime_t)p_enc->fmt_out.audio.i_rate; @@ -302,8 +304,21 @@ static block_t<br>
> *Encode( encoder_t *p_enc, block_t *p_aout_buf ) i_used =<br>
> twolame_encode_buffer_interleaved( p_sys->p_twolame, p_sys->p_buffer,<br>
> MPEG_FRAME_SIZE,<br>
>                                 p_sys->p_out_buffer, MAX_CODED_FRAME_SIZE );<br>
> +        /* On error, bufferize samples and return what was already encoded<br>
> */<br>
<br>
</span>Is bufferize even a word?<br></blockquote><div> <br></div><div>It's a function ("Bufferize") in this file, but yes I see your point.<br></div></div><br></div></div>