Hi,

Thanks for info. Could you test if attached patch helps?


W dniu 2016-05-03 o 07:57, Mahesh Pittala pisze:
Hi,

Visual Leak Detector found memory leak with this patch, seems memory is not freed up in function get_argv_utf8(). I could reproduce with Visual Studio 12(vc11) but not with Visual Studio 13(vc12), please can you check it.

encoder log - http://pastie.org/10821782
leak - http://pastie.org/10821781

On Thu, Apr 28, 2016 at 1:45 PM, Mateusz <[email protected] <mailto:[email protected]>> wrote:

    Due to problem with breaking lines I sent this patch as attachment.

    _______________________________________________
    x265-devel mailing list
    [email protected] <mailto:[email protected]>
    https://mailman.videolan.org/listinfo/x265-devel




_______________________________________________
x265-devel mailing list
[email protected]
https://mailman.videolan.org/listinfo/x265-devel

diff -r 00ea3784bd36 source/x265.cpp
--- a/source/x265.cpp   Thu Apr 28 09:59:30 2016 +0200
+++ b/source/x265.cpp   Tue May 03 12:12:12 2016 +0200
@@ -779,6 +779,10 @@
     SetConsoleTitle(orgConsoleTitle);
     SetThreadExecutionState(ES_CONTINUOUS);
 
+#if _WIN32
+    free(argv);
+#endif
+
 #if HAVE_VLD
     assert(VLDReportLeaks() == 0);
 #endif
_______________________________________________
x265-devel mailing list
[email protected]
https://mailman.videolan.org/listinfo/x265-devel

Reply via email to