Michael Niedermayer <[email protected]> added the comment:

[...]

> @@ -45,6 +46,8 @@
>  
>      s->stereo = avctx->channels > 1 ? 1 : 0;
>  
> +    s->flushed = 1;
> +
>      if ((s->gfp = lame_init()) == NULL)
>          goto err;
>      lame_set_in_samplerate(s->gfp, avctx->sample_rate);
> @@ -148,6 +151,7 @@
>      /* lame 3.91 dies on '1-channel interleaved' data */
>  
>      if(data){
> +        s->flushed = 0;
>          if (s->stereo) {
>              lame_result = lame_encode_buffer_interleaved(
>                  s->gfp,

are these 2 needed ?

[...]
> @@ -189,7 +198,7 @@
>  
>      len= mp3len(s->buffer, NULL, NULL);
>  //av_log(avctx, AV_LOG_DEBUG, "in:%d packet-len:%d index:%d\n", 
> avctx->frame_size, len, s->buffer_index);
> -    if(len <= s->buffer_index){
> +    if((len > 0) && (len <= s->buffer_index)){
>          memcpy(frame, s->buffer, len);
>          s->buffer_index -= len;
>  

why do you change this?

________________________________________________
FFmpeg issue tracker <[email protected]>
<https://roundup.ffmpeg.org/issue803>
________________________________________________

Reply via email to