[MPlayer-cvslog] r35795 - in trunk/libass: ass.c ass_parse.c ass_render.c

[Alexander Strasser]

Hi,

SubJunk wrote:
> Author: SubJunk
> Date: Sat Jan 19 22:47:31 2013
> New Revision: 35795
> 
> Log:
> Updated libass to 7e45532
> 
> - Removes a redundant warning message
> - Fixes a wrong condition
> - Fixes a leak when an error occurs
> 
> Modified:
>    trunk/libass/ass.c
>    trunk/libass/ass_parse.c
>    trunk/libass/ass_render.c
> 
> Modified: trunk/libass/ass.c
> ==============================================================================
> --- trunk/libass/ass.c	Sat Jan 19 14:00:41 2013	(r35794)
> +++ trunk/libass/ass.c	Sat Jan 19 22:47:31 2013	(r35795)
> @@ -988,7 +988,9 @@ static char *sub_recode(ASS_Library *lib
>                      oleft += size;
>                  } else {
>                      ass_msg(library, MSGL_WARN, "Error recoding file");
> -                    return NULL;
> +                    free(outbuf);

> +		    outbuf = NULL;
> +		    goto out;

  Seems to me the original did not contain that tab-space mixup. Might
be a good idea to find out how it was introduced here.

>                  }
>              } else if (clear)
>                  break;

[...]

  Should be fine otherwise.

  Alexander