Viewing Issue Advanced Details
[ Jump to Notes ]
ID Category [?] Severity [?] Reproducibility Date Submitted Last Update
06813 Misc. Minor Always Jan 1, 2018, 12:51 Jan 25, 2018, 03:26
Tester Firewave View Status Public Platform
Assigned To AJR Resolution Fixed OS
Status [?] Resolved Driver
Version 0.193 Fixed in Version 0.194 Build
Fixed in Git Commit Github Pull Request #
Summary 06813: Memory leak in options
Description 
Direct leak of 44 byte(s) in 1 object(s) allocated from:
    #0 0x14d2a23 in malloc /opt/media/clang_nightly/llvm/utils/release/final/llvm.src/projects/compiler-rt/lib/asan/asan_malloc_linux.cc:67:3
    #1 0xf18f44f in core_strdup(char const*) /mnt/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/lib/util/corestr.cpp:130:17
    #2 0x8af7b6f in osd_common_t::update_option(char const*, std::vector<char const*, std::allocator<char const*> >&) const /mnt/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/osd/modules/lib/osdobj_common.cpp:374:33
    #3 0x8af6611 in osd_common_t::register_options() /mnt/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/osd/modules/lib/osdobj_common.cpp:327:2
    #4 0x8acdde4 in main /mnt/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/osd/sdl/sdlmain.cpp:215:7
    #5 0x7f35d4ef782f in __libc_start_main (/lib/x86_64-linux-gnu/libc.so.6+0x2082f)


Direct leak of 152 byte(s) in 9 object(s) allocated from:
    #0 0x14d2a23 in malloc /opt/media/clang_nightly/llvm/utils/release/final/llvm.src/projects/compiler-rt/lib/asan/asan_malloc_linux.cc:67:3
    #1 0xf18f44f in core_strdup(char const*) /mnt/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/lib/util/corestr.cpp:130:17
    #2 0x8cdbeeb in plugin_options::parse_json(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >) /mnt/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/frontend/mame/pluginopts.cpp:78:46
    #3 0x8cda232 in plugin_options::parse_json(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >) /mnt/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/frontend/mame/pluginopts.cpp:89:6
    #4 0x8cce29d in mame_machine_manager::start_luaengine() /mnt/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/frontend/mame/mame.cpp:121:15
    #5 0x8e1dfe9 in cli_frontend::start_execution(mame_machine_manager*, std::vector<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, std::allocator<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > > > const&) /mnt/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/frontend/mame/clifront.cpp:243:11
    #6 0x8e20ee0 in cli_frontend::execute(std::vector<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, std::allocator<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > > >&) /mnt/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/frontend/mame/clifront.cpp:273:3
    #7 0x8cd3717 in emulator_info::start_frontend(emu_options&, osd_interface&, std::vector<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, std::allocator<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > > >&) /mnt/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/frontend/mame/mame.cpp:336:18
    #8 0x8acddf2 in main /mnt/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/osd/sdl/sdlmain.cpp:216:9
    #9 0x7f35d4ef782f in __libc_start_main (/lib/x86_64-linux-gnu/libc.so.6+0x2082f)
Steps To Reproduce
Additional Information
Github Commit
Flags
Regression Version
Affected Sets / Systems
Attached Files
  
Relationships
There are no relationship linked to this issue.
Notes
1
User avatar
No.14683
AJR
Developer
Jan 25, 2018, 03:26
 As of 57bd62a1fbd890d2bb546acea9c52dcb1e56a7ec, the core_strdup function has been removed from MAME. Its associated memory leaks will not be missed.