Viewing Issue Advanced Details
ID Category [?] Severity [?] Reproducibility Date Submitted Last Update
09137 Misc. Minor Always 29 days ago 26 days ago
Tester Firewave View Status Public Platform MAME (Self-compiled)
Assigned To Resolution Open OS Linux (64-bit)
Status [?] Acknowledged Driver
Version 0.275 Fixed in Version Build 64-bit
Fixed in Git Commit Github Pull Request #
Summary 09137: AddressSanitizer: new-delete-type-mismatch when selecting device in file manager
Description
==26128==ERROR: AddressSanitizer: new-delete-type-mismatch on 0x506001268180 in thread T0:
  object passed to delete has wrong type:
  size of the allocated type:   55 bytes;
  size of the deallocated type: 32 bytes.
    #0 0x5cb69f59bcb2 in operator delete(void*, unsigned long) (/home/user/CLionProjects/mame/mame-asan+0x10c58cb2) (BuildId: 54dfcfa006cf3d70783a4aced53f1e126ecd9740)
    #1 0x5cb6be56d9fe in operator() /usr/bin/../lib64/gcc/x86_64-pc-linux-gnu/14.2.1/../../../../include/c++/14.2.1/bits/unique_ptr.h:93:2
    #2 0x5cb6be56d9fe in ~unique_ptr /usr/bin/../lib64/gcc/x86_64-pc-linux-gnu/14.2.1/../../../../include/c++/14.2.1/bits/unique_ptr.h:399:4
    #3 0x5cb6be56d9fe in zippath_resolve /home/user/CLionProjects/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/lib/util/zippath.cpp:236:2
    #4 0x5cb6be56d9fe in util::zippath_directory::open(std::basic_string_view<char, std::char_traits<char>>, std::unique_ptr<util::zippath_directory, std::default_delete<util::zippath_directory>>&) /home/user/CLionProjects/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/lib/util/zippath.cpp:525:36
    #5 0x5cb6b8c443f0 in ui::menu_control_device_image::menu_control_device_image(mame_ui_manager&, render_container&, device_image_interface&) /home/user/CLionProjects/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/frontend/mame/ui/imgcntrl.cpp:121:7
    #6 0x5cb6b8c3d052 in make_unique<ui::menu_control_device_image, mame_ui_manager &, render_container &, device_image_interface &> /usr/bin/../lib64/gcc/x86_64-pc-linux-gnu/14.2.1/../../../../include/c++/14.2.1/bits/unique_ptr.h:1077:34
    #7 0x5cb6b8c3d052 in stack_push<ui::menu_control_device_image, mame_ui_manager &, render_container &, device_image_interface &> /home/user/CLionProjects/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/frontend/mame/ui/menu.h:96:14
    #8 0x5cb6b8c3d052 in ui::menu_file_manager::handle(ui::menu::event const*) /home/user/CLionProjects/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/frontend/mame/ui/filemngr.cpp:266:7
    #9 0x5cb6b8b9a2ea in ui::menu::do_handle() /home/user/CLionProjects/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/frontend/mame/ui/menu.cpp:2076:16
    #10 0x5cb6b8b97858 in ui::menu::global_state::ui_handler(render_container&) /home/user/CLionProjects/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/frontend/mame/ui/menu.cpp:255:40
    #11 0x5cb6b8ab6581 in operator() /home/user/CLionProjects/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/lib/util/delegate.h:765:11
    #12 0x5cb6b8ab6581 in mame_ui_manager::update_and_render(render_container&) /home/user/CLionProjects/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/frontend/mame/ui/ui.cpp:909:34
    #13 0x5cb6b86bbfbe in emulator_info::draw_user_interface(running_machine&) /home/user/CLionProjects/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/frontend/mame/mame.cpp:464:48
    #14 0x5cb6bcd61c15 in video_manager::frame_update(bool) /home/user/CLionProjects/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/emu/video.cpp:223:21
    #15 0x5cb6bccc463c in screen_device::vblank_begin(int) /home/user/CLionProjects/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/emu/screen.cpp:1645:21
    #16 0x5cb6bccb72dd in operator() /home/user/CLionProjects/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/lib/util/delegate.h:765:11
    #17 0x5cb6bccb72dd in device_scheduler::execute_timers() /home/user/CLionProjects/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/emu/schedule.cpp:951:5
    #18 0x5cb6bccb2180 in device_scheduler::timeslice() /home/user/CLionProjects/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/emu/schedule.cpp:504:2
    #19 0x5cb6bcb8a04b in running_machine::run(bool) /home/user/CLionProjects/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/emu/machine.cpp:333:17
    #20 0x5cb6b86b91a8 in mame_machine_manager::execute() /home/user/CLionProjects/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/frontend/mame/mame.cpp:288:19
    #21 0x5cb6b942773e 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&) /home/user/CLionProjects/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/frontend/mame/clifront.cpp:277:22
    #22 0x5cb6b942a1a4 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>>>>&) /home/user/CLionProjects/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/frontend/mame/clifront.cpp:293:3
    #23 0x5cb6b86bba3d 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>>>>&) /home/user/CLionProjects/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/frontend/mame/mame.cpp:453:18
    #24 0x5cb6bcdd0365 in main /home/user/CLionProjects/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/osd/sdl/sdlmain.cpp:100:9
    #25 0x7c2ff2e35487  (/usr/lib/libc.so.6+0x27487) (BuildId: 695cfc6aac7d0f77bb7caba0ef01b2e868762b02)
    #26 0x7c2ff2e3554b in __libc_start_main (/usr/lib/libc.so.6+0x2754b) (BuildId: 695cfc6aac7d0f77bb7caba0ef01b2e868762b02)
    #27 0x5cb69f45c124 in _start (/home/user/CLionProjects/mame/mame-asan+0x10b19124) (BuildId: 54dfcfa006cf3d70783a4aced53f1e126ecd9740)

0x506001268180 is located 0 bytes inside of 55-byte region [0x506001268180,0x5060012681b7)
allocated by thread T0 here:
    #0 0x5cb69f59ad72 in operator new(unsigned long) (/home/user/CLionProjects/mame/mame-asan+0x10c57d72) (BuildId: 54dfcfa006cf3d70783a4aced53f1e126ecd9740)
    #1 0x5cb6bf2177e3 in osd_stat(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>> const&) /home/user/CLionProjects/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/osd/modules/file/posixfile.cpp:384:59
    #2 0x5cb6be56d9ca in zippath_resolve /home/user/CLionProjects/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/lib/util/zippath.cpp:222:24
    #3 0x5cb6be56d9ca in util::zippath_directory::open(std::basic_string_view<char, std::char_traits<char>>, std::unique_ptr<util::zippath_directory, std::default_delete<util::zippath_directory>>&) /home/user/CLionProjects/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/lib/util/zippath.cpp:525:36
    #4 0x5cb6b8c443f0 in ui::menu_control_device_image::menu_control_device_image(mame_ui_manager&, render_container&, device_image_interface&) /home/user/CLionProjects/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/frontend/mame/ui/imgcntrl.cpp:121:7
    #5 0x5cb6b8c3d052 in make_unique<ui::menu_control_device_image, mame_ui_manager &, render_container &, device_image_interface &> /usr/bin/../lib64/gcc/x86_64-pc-linux-gnu/14.2.1/../../../../include/c++/14.2.1/bits/unique_ptr.h:1077:34
    #6 0x5cb6b8c3d052 in stack_push<ui::menu_control_device_image, mame_ui_manager &, render_container &, device_image_interface &> /home/user/CLionProjects/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/frontend/mame/ui/menu.h:96:14
    #7 0x5cb6b8c3d052 in ui::menu_file_manager::handle(ui::menu::event const*) /home/user/CLionProjects/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/frontend/mame/ui/filemngr.cpp:266:7
    #8 0x5cb6b8b9a2ea in ui::menu::do_handle() /home/user/CLionProjects/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/frontend/mame/ui/menu.cpp:2076:16
    #9 0x5cb6b8b97858 in ui::menu::global_state::ui_handler(render_container&) /home/user/CLionProjects/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/frontend/mame/ui/menu.cpp:255:40
    #10 0x5cb6b8ab6581 in operator() /home/user/CLionProjects/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/lib/util/delegate.h:765:11
    #11 0x5cb6b8ab6581 in mame_ui_manager::update_and_render(render_container&) /home/user/CLionProjects/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/frontend/mame/ui/ui.cpp:909:34
    #12 0x5cb6b86bbfbe in emulator_info::draw_user_interface(running_machine&) /home/user/CLionProjects/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/frontend/mame/mame.cpp:464:48
    #13 0x5cb6bcd61c15 in video_manager::frame_update(bool) /home/user/CLionProjects/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/emu/video.cpp:223:21
    #14 0x5cb6bccc463c in screen_device::vblank_begin(int) /home/user/CLionProjects/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/emu/screen.cpp:1645:21
    #15 0x5cb6bccb72dd in operator() /home/user/CLionProjects/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/lib/util/delegate.h:765:11
    #16 0x5cb6bccb72dd in device_scheduler::execute_timers() /home/user/CLionProjects/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/emu/schedule.cpp:951:5
    #17 0x5cb6bccb2180 in device_scheduler::timeslice() /home/user/CLionProjects/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/emu/schedule.cpp:504:2
    #18 0x5cb6bcb8a04b in running_machine::run(bool) /home/user/CLionProjects/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/emu/machine.cpp:333:17
    #19 0x5cb6b86b91a8 in mame_machine_manager::execute() /home/user/CLionProjects/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/frontend/mame/mame.cpp:288:19
    #20 0x5cb6b942773e 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&) /home/user/CLionProjects/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/frontend/mame/clifront.cpp:277:22
    #21 0x5cb6b942a1a4 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>>>>&) /home/user/CLionProjects/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/frontend/mame/clifront.cpp:293:3
    #22 0x5cb6b86bba3d 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>>>>&) /home/user/CLionProjects/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/frontend/mame/mame.cpp:453:18
    #23 0x5cb6bcdd0365 in main /home/user/CLionProjects/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/osd/sdl/sdlmain.cpp:100:9
    #24 0x7c2ff2e35487  (/usr/lib/libc.so.6+0x27487) (BuildId: 695cfc6aac7d0f77bb7caba0ef01b2e868762b02)
    #25 0x7c2ff2e3554b in __libc_start_main (/usr/lib/libc.so.6+0x2754b) (BuildId: 695cfc6aac7d0f77bb7caba0ef01b2e868762b02)
    #26 0x5cb69f45c124 in _start (/home/user/CLionProjects/mame/mame-asan+0x10b19124) (BuildId: 54dfcfa006cf3d70783a4aced53f1e126ecd9740)
Steps To Reproduce - run gba
- open TAB menu
- select "File Manager"
- select "cartridge (cart)"
Additional Information
Github Commit
Flags
Regression Version
Affected Sets / Systems
Attached Files
 
Relationships
There are no relationship linked to this issue.
Notes
6
User avatar
No.23096
cuavas
Administrator
28 days ago
It’s harmless. The reason you get a warning is because it allocates extra space for the path immediately following the osd::directory::entry structure. A regular delete on the pointer cleans it up correctly.
User avatar
No.23109
Robbbert
Moderator
26 days ago
Can this be closed? I saw some commits which might be related.
User avatar
No.23112
cuavas
Administrator
26 days ago
The commits are just to tidy the code a bit. It won’t make the warning go away. You can close it if you want, but even if you don’t close it, the severity definitely shouldn’t be “critical”.
User avatar
No.23113
JimCarlTay
Tester
26 days ago
Lowered severity to Major.
User avatar
No.23114
cuavas
Administrator
26 days ago
A harmless warning is not a major issue.
User avatar
No.23115
Robbbert
Moderator
26 days ago
Set to Minor, on cuavas's advice.

I'm not even sure why a Tester can modify another person's report.

I'll leave this for a week and if no further input then it can be closed.