search

runt18 / libyuv

Automatically exported from code.google.com/p/libyuv
BSD 3-Clause "New" or "Revised" License
0 stars 0 forks source link

memory leak in unittest #524

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
Error #26: POSSIBLE LEAK 230463 direct bytes 0x02ce0020-0x02d1845f + 0 indirect 
bytes
#0 replace_malloc                                                             
[d:\drmemory_package\common\alloc_replace.c:2292]
#1 libyuv::LibYUVConvertTest_I420AlphaToABGR_ARGB_Premult_Test::TestBody      
[d:\src\libyuv\libyuv\unit_test\convert_test.cc:1867]
#2 testing::internal::HandleExceptionsInMethodIfSupported<testing::Test,void> 
[d:\src\libyuv\libyuv\testing\gtest\src\gtest.cc:2458]
#3 testing::Test::Run                                                         
[d:\src\libyuv\libyuv\testing\gtest\src\gtest.cc:2475]
#4 testing::TestInfo::Run                                                     
[d:\src\libyuv\libyuv\testing\gtest\src\gtest.cc:2656]
#5 testing::TestCase::Run                                                     
[d:\src\libyuv\libyuv\testing\gtest\src\gtest.cc:2774]
#6 testing::internal::UnitTestImpl::RunAllTests                               
[d:\src\libyuv\libyuv\testing\gtest\src\gtest.cc:4647]
#7 
testing::internal::HandleExceptionsInMethodIfSupported<testing::internal::UnitTe
stImpl,bool> [d:\src\libyuv\libyuv\testing\gtest\src\gtest.cc:2458]
#8 testing::UnitTest::Run                                                     
[d:\src\libyuv\libyuv\testing\gtest\src\gtest.cc:4258]
#9 RUN_ALL_TESTS                                                              
[d:\src\libyuv\libyuv\testing\gtest\include\gtest\gtest.h:2237]
#10 main                                                                       
[d:\src\libyuv\libyuv\unit_test\unit_test.cc:355]

Original issue reported on code.google.com by fbarch...@chromium.org on 17 Nov 2015 at 8:28

GoogleCodeExporter commented 8 years ago 
The following revision refers to this bug:
  https://chromium.googlesource.com/libyuv/libyuv.git/+/ec4b258d4e5a2ab3efaee4bfbfd8fb5494c89112

commit ec4b258d4e5a2ab3efaee4bfbfd8fb5494c89112
Author: Frank Barchard <fbarchard@google.com>
Date: Tue Nov 17 08:29:53 2015

free src_a in unittest to fix leak

TBR=harryjin@google.com
BUG=libyuv:524

Review URL: https://codereview.chromium.org/1452083002 .

[modify] 
http://crrev.com/ec4b258d4e5a2ab3efaee4bfbfd8fb5494c89112/README.chromium
[modify] 
http://crrev.com/ec4b258d4e5a2ab3efaee4bfbfd8fb5494c89112/include/libyuv/version
.h
[modify] 
http://crrev.com/ec4b258d4e5a2ab3efaee4bfbfd8fb5494c89112/unit_test/convert_test
.cc

Original comment by bugdroid1@chromium.org on 17 Nov 2015 at 8:31

GoogleCodeExporter commented 8 years ago

Original comment by fbarch...@chromium.org on 17 Nov 2015 at 8:32

GoogleCodeExporter commented 8 years ago 
The following revision refers to this bug:
  https://chromium.googlesource.com/libyuv/libyuv.git/+/526558b2d86ecfa6408953ea53da107f15211197

commit 526558b2d86ecfa6408953ea53da107f15211197
Author: Frank Barchard <fbarchard@google.com>
Date: Thu Nov 19 10:25:00 2015

disable debug build of 411 to work around compiler bug

TBR=harryjin@google.com
BUG=libyuv:524

Review URL: https://codereview.chromium.org/1461013002 .

[modify] 
http://crrev.com/526558b2d86ecfa6408953ea53da107f15211197/README.chromium
[modify] 
http://crrev.com/526558b2d86ecfa6408953ea53da107f15211197/include/libyuv/row.h
[modify] 
http://crrev.com/526558b2d86ecfa6408953ea53da107f15211197/include/libyuv/version
.h
[modify] 
http://crrev.com/526558b2d86ecfa6408953ea53da107f15211197/source/row_any.cc
[modify] 
http://crrev.com/526558b2d86ecfa6408953ea53da107f15211197/source/row_gcc.cc

Original comment by bugdroid1@chromium.org on 19 Nov 2015 at 10:26

GoogleCodeExporter commented 8 years ago 
The following revision refers to this bug:
  https://chromium.googlesource.com/libyuv/libyuv.git/+/88552486f1322b5d4da9ea7f833ac60998331f5d

commit 88552486f1322b5d4da9ea7f833ac60998331f5d
Author: Frank Barchard <fbarchard@google.com>
Date: Fri Nov 20 19:21:39 2015

disable 411 on x86 due to compile error

TBR=harryjin@google.com
BUG=libyuv:524

Review URL: https://codereview.chromium.org/1468523002 .

[modify] 
http://crrev.com/88552486f1322b5d4da9ea7f833ac60998331f5d/README.chromium
[modify] 
http://crrev.com/88552486f1322b5d4da9ea7f833ac60998331f5d/include/libyuv/row.h
[modify] 
http://crrev.com/88552486f1322b5d4da9ea7f833ac60998331f5d/include/libyuv/version
.h

Original comment by bugdroid1@chromium.org on 20 Nov 2015 at 7:22