On 07/21/17 09:29, [email protected] wrote:
# HG changeset patch
# User Aruna Matheswaran <[email protected]>
# Date 1500457328 -19800
# Wed Jul 19 15:12:08 2017 +0530
# Branch stable
# Node ID fd354d5ec1328a000c24e1551804a1ce569ec3b0
# Parent adbcc90bdef36b50a091deb5b0d0ad77debfbee7
threadpool: fix memory leak
diff -r adbcc90bdef3 -r fd354d5ec132 source/common/threadpool.cpp
--- a/source/common/threadpool.cpp Thu Jul 13 16:50:18 2017 +0530
+++ b/source/common/threadpool.cpp Wed Jul 19 15:12:08 2017 +0530
@@ -454,6 +454,7 @@
if ((nodeMaskPerPool[node] >> j) & 1)
len += sprintf(nodesstr + len, ",%d", j);
x265_log(p, X265_LOG_INFO, "Thread pool %d using %d threads on
numa nodes %s\n", i, numThreads, nodesstr + 1);
+ free(nodesstr);
This applies free() to a pointer obtained with new[]. It should be
delete[] or better yet - replace the original new with a buffer on the
stack.
}
else
x265_log(p, X265_LOG_INFO, "Thread pool created using %d
threads\n", numThreads);
_______________________________________________
x265-devel mailing list
[email protected]
https://mailman.videolan.org/listinfo/x265-devel
