[FFmpeg-devel] [PATCH] libavformat/dashdec: Fix for Bug #7338

Carl Eugen Hoyos ceffmpeg at gmail.com
Tue Aug 7 23:22:38 EEST 2018


2018-08-07 22:12 GMT+02:00, Colin NG <colin_ng at hotmail.com>:
> ---
>  libavformat/dashdec.c | 4 +---
>  1 file changed, 1 insertion(+), 3 deletions(-)
>
> diff --git a/libavformat/dashdec.c b/libavformat/dashdec.c
> index 09b06ef..fa784f1 100644
> --- a/libavformat/dashdec.c
> +++ b/libavformat/dashdec.c
> @@ -1040,7 +1040,6 @@ end:
>          xmlFree(rep_bandwidth_val);
>      if (rep_framerate_val)
>          xmlFree(rep_framerate_val);
> -
>      return ret;
>  }

No unrelated changes please.

> @@ -1869,7 +1868,7 @@ static int is_common_init_section_exist(struct
> representation **pls, int n_pls)
>      url = first_init_section->url;
>      url_offset = first_init_section->url_offset;
>      size = pls[0]->init_section->size;
> -    for (i=0;i<n_pls;i++) {
> +    for (i=1; i<n_pls; i++) {

If you change the whitespace here (many will find this is a good idea),
please also add spaces around the operators: (i = 1; i < n_pls; i++)

Carl Eugen


More information about the ffmpeg-devel mailing list