Viewing Issue Advanced Details
ID Category [?] Severity [?] Reproducibility Date Submitted Last Update
08472 Misc. Critical (emulator) Sometimes Oct 28, 2022, 11:25 Nov 4, 2022, 13:55
Tester Firewave View Status Public Platform MAME (Official Binary)
Assigned To Resolution Open OS Windows 10/11 (64-bit)
Status [?] Confirmed Driver
Version 0.249 Fixed in Version Build 64-bit
Fixed in Git Commit Github Pull Request #
Summary 08472: some sinclair/* drivers: Executable might exit with -1073740940
Description
C:\Apps\mame>mame -window atmtb2plus -str 2
Average speed: 100.51% (1 seconds)

C:\Apps\mame>echo %ERRORLEVEL%
-1073740940

There's no issue running it at all. It is just the exitcode.

This exitcode might also appear as 0xC0000374 in other contexts. I have not found a proper description of this error but it appears to indicate a "heap corruption".
Steps To Reproduce
Additional Information Affected sets:

sinclair/atm.cpp
atmtb2
atmtb2plus

sinclair/pentevo.cpp
pentevo
Github Commit
Flags
Regression Version
Affected Sets / Systems some sinclair/* drivers
Attached Files
 
Relationships
There are no relationship linked to this issue.
Notes
4
User avatar
No.20675
Robbbert
Senior Tester
Oct 29, 2022, 15:43
Confirmed, but only for the sinclair machines (atmtb2, atmtb2plus, pentevo). The exidy games always returned 0 for me.
User avatar
No.20678
Firewave
Senior Tester
Oct 29, 2022, 17:55
I had the wrong reproducibility set. It does not happen all the time - but it is limited to those drivers only.
User avatar
No.20679
holub
Tester
Oct 30, 2022, 06:14
Not sure what is exect cause but a specific of theses devices is usage of m_screen->configure. Screen size selected can be bigger than configured on startup.
User avatar
No.20681
Firewave
Senior Tester
Nov 1, 2022, 23:47
edited on: Nov 2, 2022, 22:54
0.249 reports this for atmtb2, atmtb2plus, pentevo
=================================================================
==30316==ERROR: AddressSanitizer: heap-buffer-overflow on address 0x602000102476 at pc 0x7f0db2b39b07 bp 0x7fffcf1787e0 sp 0x7fffcf1787d8
WRITE of size 1 at 0x602000102476 thread T0
    #0 0x7f0db2b39b06 in atm_state::atm_port_ff_w(unsigned int, unsigned char) /mnt/s/GitHub/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/mame/sinclair/atm.cpp:98:23
    #1 0x7f0dc38a559c in operator() /mnt/s/GitHub/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/lib/util/delegate.h:765:11
    #2 0x7f0dc38a559c in std::enable_if<(((std::is_same<emu::device_delegate<void (unsigned int, unsigned char)>, emu::device_delegate<void (unsigned int, unsigned char)> >::value) || (std::is_same<emu::device_delegate<void (unsigned int, unsigned char)>, emu::device_delegate<void (unsigned int, unsigned short)> >::value)) || (std::is_same<emu::device_delegate<void (unsigned int, unsigned char)>, emu::device_delegate<void (unsigned int, unsigned int)> >::value)) || (std::is_same<emu::device_delegate<void (unsigned int, unsigned char)>, emu::device_delegate<void (unsigned int, unsigned long)> >::value), void>::type handler_entry_write_delegate<0, 0, emu::device_delegate<void (unsigned int, unsigned char)> >::write_impl<emu::device_delegate<void (unsigned int, unsigned char)> >(unsigned int, unsigned char, unsigned char) const /mnt/s/GitHub/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/emu/emumem_hedp.cpp:125:2
    #3 0x7f0dc38a5418 in handler_entry_write_delegate<0, 0, emu::device_delegate<void (unsigned int, unsigned char)> >::write(unsigned int, unsigned char, unsigned char) const /mnt/s/GitHub/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/emu/emumem_hedp.cpp:150:2
    #4 0x7f0db4f0fa58 in void dispatch_write<0, 0, 0>(unsigned int, unsigned int, emu::detail::handler_entry_size<0>::uX, emu::detail::handler_entry_size<0>::uX, handler_entry_write<0, 0> const* const*) /mnt/s/GitHub/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/emu/emumem.h:1577:47
    #5 0x7f0dc67bef31 in handler_entry_write_dispatch<14, 0, 0>::write(unsigned int, unsigned char, unsigned char) const /mnt/s/GitHub/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/emu/emumem_hedw.ipp:131:2
    #6 0x7f0da9e6f95b in void dispatch_write<1, 0, 0>(unsigned int, unsigned int, emu::detail::handler_entry_size<0>::uX, emu::detail::handler_entry_size<0>::uX, handler_entry_write<0, 0> const* const*) /mnt/s/GitHub/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/emu/emumem.h:1577:47
    #7 0x7f0dc67e9741 in handler_entry_write_dispatch<16, 0, 0>::write(unsigned int, unsigned char, unsigned char) const /mnt/s/GitHub/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/emu/emumem_hedw.ipp:131:2
    #8 0x7f0da9e6f95b in void dispatch_write<1, 0, 0>(unsigned int, unsigned int, emu::detail::handler_entry_size<0>::uX, emu::detail::handler_entry_size<0>::uX, handler_entry_write<0, 0> const* const*) /mnt/s/GitHub/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/emu/emumem.h:1577:47
    #9 0x7f0da9e6e994 in write_native /mnt/s/GitHub/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/emu/emumem.h:1741:3
    #10 0x7f0da9e6e994 in emu::detail::memory_access_specific<1, 0, 0, (util::endianness)0>::write_byte(unsigned int, unsigned char) /mnt/s/GitHub/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/emu/emumem.h:1657:70
    #11 0x7f0dbd50ae50 in z80_device::out(unsigned short, unsigned char) /mnt/s/GitHub/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/devices/cpu/z80/z80.cpp:457:7
    #12 0x7f0dbd4d25fd in z80_device::op_d3() /mnt/s/GitHub/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/devices/cpu/z80/z80.cpp:3174:44
    #13 0x7f0dbd4b41a4 in z80_device::execute_run() /mnt/s/GitHub/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/devices/cpu/z80/z80.cpp:3654:3
    #14 0x7f0dbd4b6bff in non-virtual thunk to z80_device::execute_run() /mnt/s/GitHub/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/devices/cpu/z80/z80.cpp
    #15 0x7f0dc9022817 in run /mnt/s/GitHub/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/emu/diexec.h:190:15
    #16 0x7f0dc9022817 in device_scheduler::timeslice() /mnt/s/GitHub/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/emu/schedule.cpp:456:14
    #17 0x7f0dc8ebb4a7 in running_machine::run(bool) /mnt/s/GitHub/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/emu/machine.cpp:329:17
    #18 0x7f0dcc021f7f in mame_machine_manager::execute() /mnt/s/GitHub/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/frontend/mame/mame.cpp:290:19
    #19 0x7f0dcc2168d6 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/s/GitHub/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/frontend/mame/clifront.cpp:275:22
    #20 0x7f0dcc21a41f 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/s/GitHub/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/frontend/mame/clifront.cpp:291:3
    #21 0x7f0dcc026d5f 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/s/GitHub/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/frontend/mame/mame.cpp:454:18
    #22 0x7f0dc91fd58b in main /mnt/s/GitHub/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/osd/sdl/sdlmain.cpp:191:9
    #23 0x7f0d876f9209 in __libc_start_call_main csu/../sysdeps/nptl/libc_start_call_main.h:58:16
    #24 0x7f0d876f92bb in __libc_start_main csu/../csu/libc-start.c:389:3
    #25 0x7f0da6987260 in _start (/mnt/s/GitHub/mame/mame+0x1d397260) (BuildId: 603d3d1c300651feb2a8e3ac6e9cb58d3f85e77b)

0x602000102476 is located 5 bytes to the right of 1-byte region [0x602000102470,0x602000102471)
allocated by thread T0 here:
    #0 0x7f0da6a44e7d in operator new(unsigned long) (/mnt/s/GitHub/mame/mame+0x1d454e7d) (BuildId: 603d3d1c300651feb2a8e3ac6e9cb58d3f85e77b)
    #1 0x7f0da7018742 in allocate /usr/bin/../lib/gcc/x86_64-linux-gnu/12/../../../../include/c++/12/bits/new_allocator.h:137:27
    #2 0x7f0da7018742 in allocate /usr/bin/../lib/gcc/x86_64-linux-gnu/12/../../../../include/c++/12/bits/alloc_traits.h:464:20
    #3 0x7f0da7018742 in std::_Vector_base<unsigned char, std::allocator<unsigned char> >::_M_allocate(unsigned long) /usr/bin/../lib/gcc/x86_64-linux-gnu/12/../../../../include/c++/12/bits/stl_vector.h:378:20
    #4 0x7f0db2b4d7b4 in unsigned char* std::vector<unsigned char, std::allocator<unsigned char> >::_M_allocate_and_copy<unsigned char const*>(unsigned long, unsigned char const*, unsigned char const*) /usr/bin/../lib/gcc/x86_64-linux-gnu/12/../../../../include/c++/12/bits/stl_vector.h:1614:29
    #5 0x7f0db2b4d3b4 in void std::vector<unsigned char, std::allocator<unsigned char> >::_M_assign_aux<unsigned char const*>(unsigned char const*, unsigned char const*, std::forward_iterator_tag) /usr/bin/../lib/gcc/x86_64-linux-gnu/12/../../../../include/c++/12/bits/vector.tcc:318:20
    #6 0x7f0db2b4afae in std::vector<unsigned char, std::allocator<unsigned char> >::operator=(std::initializer_list<unsigned char>) /usr/bin/../lib/gcc/x86_64-linux-gnu/12/../../../../include/c++/12/bits/stl_vector.h:785:8
    #7 0x7f0db2b4358a in atm_state::machine_reset() /mnt/s/GitHub/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/mame/sinclair/atm.cpp:447:17
    #8 0x7f0dc284a0b0 in driver_device::device_reset_after_children() /mnt/s/GitHub/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/emu/driver.cpp:248:3
    #9 0x7f0dc2694880 in device_t::reset() /mnt/s/GitHub/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/emu/device.cpp:355:2
    #10 0x7f0dc8eb9854 in running_machine::reset_all_devices() /mnt/s/GitHub/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/emu/machine.cpp:1040:16
    #11 0x7f0dc8ebfc29 in operator() /mnt/s/GitHub/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/lib/util/delegate.h:765:11
    #12 0x7f0dc8ebfc29 in running_machine::call_notifiers(machine_notification) /mnt/s/GitHub/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/emu/machine.cpp:849:3
    #13 0x7f0dc8eb92c8 in running_machine::soft_reset(int) /mnt/s/GitHub/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/emu/machine.cpp:956:2
    #14 0x7f0dc8ebb35f in running_machine::run(bool) /mnt/s/GitHub/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/emu/machine.cpp:309:3
    #15 0x7f0dcc021f7f in mame_machine_manager::execute() /mnt/s/GitHub/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/frontend/mame/mame.cpp:290:19
    #16 0x7f0dcc2168d6 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/s/GitHub/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/frontend/mame/clifront.cpp:275:22
    #17 0x7f0dcc21a41f 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/s/GitHub/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/frontend/mame/clifront.cpp:291:3
    #18 0x7f0dcc026d5f 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/s/GitHub/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/frontend/mame/mame.cpp:454:18
    #19 0x7f0dc91fd58b in main /mnt/s/GitHub/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/osd/sdl/sdlmain.cpp:191:9
    #20 0x7f0d876f9209 in __libc_start_call_main csu/../sysdeps/nptl/libc_start_call_main.h:58:16

SUMMARY: AddressSanitizer: heap-buffer-overflow /mnt/s/GitHub/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/mame/sinclair/atm.cpp:98:23 in atm_state::atm_port_ff_w(unsigned int, unsigned char)
Shadow bytes around the buggy address:
  0x0c0480018430: fa fa fd fd fa fa fd fa fa fa fd fd fa fa fd fa
  0x0c0480018440: fa fa fd fd fa fa fd fa fa fa fd fa fa fa fd fa
  0x0c0480018450: fa fa fd fd fa fa fd fa fa fa fd fa fa fa fd fa
  0x0c0480018460: fa fa fd fd fa fa fd fa fa fa fd fa fa fa fd fa
  0x0c0480018470: fa fa fd fd fa fa fd fa fa fa fd fd fa fa fd fd
=>0x0c0480018480: fa fa fd fa fa fa fd fa fa fa 00 00 fa fa[01]fa
  0x0c0480018490: fa fa fd fd fa fa fd fa fa fa fd fa fa fa fd fa
  0x0c04800184a0: fa fa fd fd fa fa fd fd fa fa 00 00 fa fa fd fa
  0x0c04800184b0: fa fa fd fa fa fa fd fd fa fa fd fa fa fa 00 04
  0x0c04800184c0: fa fa 00 04 fa fa 00 00 fa fa fd fa fa fa fd fa
  0x0c04800184d0: fa fa 00 04 fa fa 00 04 fa fa 00 00 fa fa 00 00
Shadow byte legend (one shadow byte represents 8 application bytes):
  Addressable:           00
  Partially addressable: 01 02 03 04 05 06 07
  Heap left redzone:       fa
  Freed heap region:       fd
  Stack left redzone:      f1
  Stack mid redzone:       f2
  Stack right redzone:     f3
  Stack after return:      f5
  Stack use after scope:   f8
  Global redzone:          f9
  Global init order:       f6
  Poisoned by user:        f7
  Container overflow:      fc
  Array cookie:            ac
  Intra object redzone:    bb
  ASan internal:           fe
  Left alloca redzone:     ca
  Right alloca redzone:    cb
==30316==ABORTING