Status: Untriaged
Owner: [email protected]
Labels: Type-Bug Pri-2 OS-Linux Area-BrowserBackend Size-Medium OS-Mac

New issue 16096 by [email protected]: Thread name leak in  
PlatformThread::SetName
http://code.google.com/p/chromium/issues/detail?id=16096

the posix implementation of SetName is a stub that leaks its argument?

Very common in ui tests.   (Why isn't there a unit test for this?)
To reproduce, do something like
sh tools/valgrind/chrome_test.sh -t ui --gtest_filter=OmniboxTest.Measure

valgrind complains

33 bytes in 1 blocks are definitely lost in loss record 526 of 1,324
    at operator new(unsigned int) (vg_replace_malloc.c:212)
    by std::string::_Rep::_S_create(unsigned int, unsigned int,  
std::allocator<char> const&)
(new_allocator.h:92)
    by std::string::_Rep::_M_clone(std::allocator<char> const&, unsigned int)
(basic_string.tcc:604)
    by std::string::reserve(unsigned int) (basic_string.tcc:485)
    by std::string::append(char const*, unsigned int) (basic_string.tcc:306)
    by void StringAppendVT<std::string>(std::string*,  
std::string::value_type const*, char*)
(string_util.cc:928)
    by StringAppendV(std::string*, char const*, char*) (string_util.cc:1097)
    by StringPrintf(char const*, ...) (string_util.cc:1108)
    by (anonymous namespace)::WorkerThread::ThreadMain()  
(worker_pool_linux.cc:71)

--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--~--~---------~--~----~------------~-------~--~----~
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/group/chromium-bugs
-~----------~----~----~----~------~----~------~--~---

Reply via email to