[FFmpeg-devel] [PATCH] avformat/http: remove never twice executable loop

Tomas Härdin tomas.hardin at codemill.se
Wed May 14 09:26:52 CEST 2014


On lör, 2014-05-10 at 20:56 +0200, Michael Niedermayer wrote:
> Fixes CID1197069
> 
> Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
> ---
>  libavformat/http.c |    3 ---
>  1 file changed, 3 deletions(-)
> 
> diff --git a/libavformat/http.c b/libavformat/http.c
> index 64ca4ae..3e172e3 100644
> --- a/libavformat/http.c
> +++ b/libavformat/http.c
> @@ -890,7 +890,6 @@ static int http_read_stream(URLContext *h, uint8_t *buf, int size)
>          if (!s->chunksize) {
>              char line[32];
>  
> -            for(;;) {
>                  do {
>                      if ((err = http_get_line(s, line, sizeof(line))) < 0)
>                          return err;
> @@ -902,8 +901,6 @@ static int http_read_stream(URLContext *h, uint8_t *buf, int size)
>  
>                  if (!s->chunksize)
>                      return 0;
> -                break;
> -            }
>          }
>          size = FFMIN(size, s->chunksize);
>      }

Looks OK to me

/Tomas



More information about the ffmpeg-devel mailing list