search

thpatch / thcrap

Touhou Community Reliant Automatic Patcher
https://www.thpatch.net
The Unlicense
551 stars 41 forks source link

[SWR/soku] Memory access violation #191

Closed Calio-Templar closed 1 year ago

Calio-Templar commented 2 years ago

thcrap_log.1.txt

Calio-Templar commented 2 years ago

https://drive.google.com/file/d/1Ckrk1oXNExvP4smqJvlNVdY-A4wGTfxN/view?usp=sharing th123 dump on error

Calio-Templar commented 2 years ago

To reproduce: install korean patch, and enter story mode

brliron commented 2 years ago

It seems to work for me. Does it happen every time, or only once in a while? Does it happen on any scenario/difficulty, or only a specific one?

Calio-Templar commented 2 years ago

it happens everytime

Calio-Templar commented 1 year ago

Update, changing windows locale to utf-8 solved this issue, not sure why though

32th-System commented 1 year ago

What was your locale before?

Calio-Templar commented 1 year ago

Probably korean (cp949)

2023년 4월 3일 (월) 오전 5:49, 32th System @.***>님이 작성:

What was your locale before?

— Reply to this email directly, view it on GitHub https://github.com/thpatch/thcrap/issues/191#issuecomment-1493436907, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADCCN2BJXVBPPWUGV444DMDW7HQ6VANCNFSM543VFXEA . You are receiving this because you authored the thread.Message ID: @.***>

brliron commented 1 year ago

The crash happens in fn_for_build, with fn being that thing:

1728D2E0 74 68 31 30 35 2F 64 61 74 61 2F 62 61 74 74 6C th105/data/battl
1728D2F0 65 2F 3F E5 92 B2 EF BD BD EF BD AE 3F 3F 62 6D e/?å.²ï½½ï½®??bm
1728D300 70 00 37 37 37 37 C8 37 3E C6 7D 6A A1 03 00 80 p.7777È7>Æ}j¡...

This crash is fixed by https://github.com/thpatch/thcrap/commit/6b47a26800559312c66279377802ea3af3ce18d8 (which isn't included in the last release yet), but skimming through the logs, I've still been able to find a filename that looks like garbage (PNG) Resolving th105/data/battle/”z’u}.png...not found (full log file: thcrap_log.txt ) We should investigate it.