Viewing Issue Advanced Details
[ Jump to Notes ]
sony/psx.cpp
ID Category [?] Severity [?] Reproducibility Date Submitted Last Update
07365 Crash/Freeze Critical (emulator) Always Jun 23, 2019, 22:38 Nov 15, 2022, 10:23
Tester Kale View Status Public Platform MAME (Official Binary)
Assigned To Resolution Open OS Windows 10 (64-bit)
Status [?] Confirmed Driver
sony/psx.cpp
Version 0.210 Fixed in Version Build 64-bit
Fixed in Git Commit Github Pull Request #
Summary MESS-specific 07365: psj [senkai]: Crashes MAME after intro FMV
Description Senkai Taisen - TV Animation Senkaiden Houshin Engi Yori crashes MAME after the intro FMV, either by skipping it or not.
Steps To Reproduce Start a new game then either watch the FMV or skip it by pressing start
Additional Information
Github Commit
Flags
Regression Version
Affected Sets / Systems psj [senkai]
Attached Files
  
Relationships
There are no relationship linked to this issue.
Notes
2
User avatar
No.16586
Tafoid
Administrator
Jun 24, 2019, 22:45
-----------------------------------------------------
Exception at EIP=0000000004d77430 (psxgpu_device::update_screen(screen_device&,bitmap_rgb32&, rectangle const&)+0x01b0): ACCESS VIOLATION
While attempting to read memory at 000000000000080c
-----------------------------------------------------
  0000000000239090: 0000000004d77430 (psxgpu_device::update_screen(screen_device&, bitmap_rgb32&, rectangle const&)+0x01b0)
  00000000002391c0: 0000000005344944 (screen_device::update_partial(int)+0x0474)
  0000000000239240: 000000000536d74a (video_manager::finish_screen_updates()+0x006a)
  00000000002392c0: 000000000536fae9 (video_manager::frame_update(bool)+0x01b9)
  0000000000239320: 00000000053414c7 (screen_device::vblank_begin()+0x01b7)
  0000000000239380: 0000000005344a87 (screen_device::device_timer(emu_timer&, unsigned int, int, void*)+0x0117)
User avatar
No.20826
Firewave
Senior Tester
Nov 15, 2022, 10:23
edited on: Dec 5, 2022, 20:07
 This set is flagged MNW.

The CHD I used did not match the current software list entry. It might have been an earlier version.

0.249 on Linux reports:
==21808==ERROR: AddressSanitizer: heap-buffer-overflow on address 0x7f623b0e180a at pc 0x7f62972ad653 bp 0x7fffcda371d0 sp 0x7fffcda371c8
READ of size 2 at 0x7f623b0e180a thread T0
    #0 0x7f62972ad652 in psxgpu_device::update_screen(screen_device&, bitmap_rgb32&, rectangle const&) /mnt/s/GitHub/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/devices/video/psx.cpp:780:24
    #1 0x7f62950136bd in util::detail::delegate_base<delegate_late_bind, unsigned int, screen_device&, bitmap_rgb32&, rectangle const&>::operator()(screen_device&, bitmap_rgb32&, rectangle const&) const /mnt/s/GitHub/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/lib/util/delegate.h:765:11
    #2 0x7f629fce7571 in screen_device::update_partial(int) /mnt/s/GitHub/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/emu/screen.cpp:1216:41
    #3 0x7f629fde7719 in video_manager::finish_screen_updates() /mnt/s/GitHub/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/emu/video.cpp:622:10
    #4 0x7f629fde651b in video_manager::frame_update(bool) /mnt/s/GitHub/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/emu/video.cpp:215:44
    #5 0x7f629fcddb38 in screen_device::vblank_begin(int) /mnt/s/GitHub/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/emu/screen.cpp:1646:21
    #6 0x7f629fcc6674 in operator() /mnt/s/GitHub/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/lib/util/delegate.h:765:11
    #7 0x7f629fcc6674 in device_scheduler::execute_timers() /mnt/s/GitHub/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/emu/schedule.cpp:951:5
    #8 0x7f629fcc15b8 in device_scheduler::timeslice() /mnt/s/GitHub/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/emu/schedule.cpp:505:2
    #9 0x7f629fb5e027 in running_machine::run(bool) /mnt/s/GitHub/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/emu/machine.cpp:329:17
    #10 0x7f6297c6dc6f in mame_machine_manager::execute() /mnt/s/GitHub/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/frontend/mame/mame.cpp:290:19
    #11 0x7f6299024fe6 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
    #12 0x7f6299028b2f 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
    #13 0x7f6297c72a4f 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
    #14 0x7f629fe53d0b in main /mnt/s/GitHub/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/osd/sdl/sdlmain.cpp:191:9
    #15 0x7f6252b99209 in __libc_start_call_main csu/../sysdeps/nptl/libc_start_call_main.h:58:16
    #16 0x7f6252b992bb in __libc_start_main csu/../csu/libc-start.c:389:3
    #17 0x7f62797e4bd0 in _start (/mnt/s/GitHub/mame/mame+0x24d5fbd0) (BuildId: 5ea94812d72bae4c)

0x7f623b0e180a is located 10 bytes to the right of 1048576-byte region [0x7f623afe1800,0x7f623b0e1800)
allocated by thread T0 here:
    #0 0x7f62798a297d in operator new[](unsigned long) (/mnt/s/GitHub/mame/mame+0x24e1d97d) (BuildId: 5ea94812d72bae4c)
    #1 0x7f62972a4367 in make_unique_clear<unsigned short[]> /mnt/s/GitHub/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/lib/util/corealloc.h:42:23
    #2 0x7f62972a4367 in psxgpu_device::psx_gpu_init(int) /mnt/s/GitHub/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/devices/video/psx.cpp:516:11
    #3 0x7f62972a3bb4 in psxgpu_device::device_start() /mnt/s/GitHub/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/devices/video/psx.cpp
    #4 0x7f62992cf790 in device_t::start() /mnt/s/GitHub/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/emu/device.cpp:562:2
    #5 0x7f629fb5ce5f in running_machine::start_all_devices() /mnt/s/GitHub/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/emu/machine.cpp:1013:13
    #6 0x7f629fb5ae34 in running_machine::start() /mnt/s/GitHub/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/emu/machine.cpp:211:2
    #7 0x7f629fb5dc5c in running_machine::run(bool) /mnt/s/GitHub/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/emu/machine.cpp:281:3
    #8 0x7f6297c6dc6f in mame_machine_manager::execute() /mnt/s/GitHub/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/frontend/mame/mame.cpp:290:19
    #9 0x7f6299024fe6 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
    #10 0x7f6299028b2f 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
    #11 0x7f6297c72a4f 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
    #12 0x7f629fe53d0b in main /mnt/s/GitHub/mame/build/projects/sdl/mame/gmake-linux-clang/../../../../../src/osd/sdl/sdlmain.cpp:191:9
    #13 0x7f6252b99209 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/devices/video/psx.cpp:780:24 in psxgpu_device::update_screen(screen_device&, bitmap_rgb32&, rectangle const&)
Shadow bytes around the buggy address:
  0x0fecc76142b0: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
  0x0fecc76142c0: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
  0x0fecc76142d0: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
  0x0fecc76142e0: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
  0x0fecc76142f0: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
=>0x0fecc7614300: fa[fa]fa fa fa fa fa fa fa fa fa fa fa fa fa fa
  0x0fecc7614310: fa fa fa fa fa fa fa fa fa fa fa fa fa fa fa fa
  0x0fecc7614320: fa fa fa fa fa fa fa fa fa fa fa fa fa fa fa fa
  0x0fecc7614330: fa fa fa fa fa fa fa fa fa fa fa fa fa fa fa fa
  0x0fecc7614340: fa fa fa fa fa fa fa fa fa fa fa fa fa fa fa fa
  0x0fecc7614350: fa fa fa fa fa fa fa fa fa fa fa fa fa fa fa fa
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