summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorwm4 <wm4@nowhere>2014-11-22 03:18:25 +0100
committerwm4 <wm4@nowhere>2014-11-22 03:18:25 +0100
commit92c8055bb11b553319830703faf60de15a6a1bc1 (patch)
tree73997143851c408d2ae7930642f776e6eb8c6295
parent2dd1e11d1b3e28915f3862fb5cd2e967fee32846 (diff)
downloadlibass-92c8055bb11b553319830703faf60de15a6a1bc1.tar.bz2
libass-92c8055bb11b553319830703faf60de15a6a1bc1.tar.xz
Fix broken memory leak check in legacy API
This was a check for the realloc() call, but the result was never used in the success case - which makes no sense. Doesn't really matter, because it's the legacy charset code which hopefully is not used by anyone. Found by Coverity.
-rw-r--r--libass/ass.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/libass/ass.c b/libass/ass.c
index 75f646f..04ce895 100644
--- a/libass/ass.c
+++ b/libass/ass.c
@@ -1002,6 +1002,7 @@ static char *sub_recode(ASS_Library *library, char *data, size_t size,
outbuf = 0;
goto out;
}
+ outbuf = nbuf;
op = outbuf + offset;
osize += size;
oleft += size;