Патч игры «Аладдин» для Windows 95

Offtop

AxXxB, я про вызов "004056FA call cs:GetTickCount => call sub_41CE40" - там же адрес заменяется из таблицы релокаций, и он приходится на 2-й байт адреса нового call.

Разве что ОС вообще замену не делает, если базовый адрес совпадает, но везде ли это будет работать?

У iunnamed заменённый вызов был относительный.

 

Val07og, спасибо! Удалил из таблицы запись об этом адресе.

 

Уважаемые товарищи модераторы, может, всё, начиная с моего первого сообщения, куда-нибудь в отдельную тему вынести, чтобы людей не смущать?
А то к запуску игры программные вопросы имеют уже слабое отношение - надо было мне сразу отдельную тему создать где-нибудь в "Мастерской".

---------- Сообщение добавлено в 02:49 ---------- Предыдущее сообщение размещено в 02:44 ----------

Пока не починим окончательно.

 

Offtop

Хм, а ведь вызовы GetTickCount в новой функции абсолютные. Надо бы относительными сделать.

 

А если один из двух GetTickCount (там вроде можно одним обойтись) заменить на Sleep(14)?
upd. Понял, тогда импорт новый придётся добавлять (способ, предложенный iunnamed, мне не очень нравится тем, что 60 раз в секунду будет загружаться/освобождаться библиотека).

AxXxB, а относительные - это примерно так?

Код:

...
  call _newProc ; заменяющий вызов
_addr1:
...
; где-то по коду
_addr2:
  call cs:GetTickCount ; абсолютный
...

_newProc:
pop eax
mov eax,[eax+_addr2-_addr1+3]
call eax
...
jmp _addr1

Торможу, там же jmp ds:GetTickCount есть в 41CC4E.

upd2. AxXxB, и всё-таки без Sleep, наверное, не обойтись. Представляешь, с какой частотой будет вызываться GetTickCount в этом цикле ожидания? Получится совершенно ненужная загрузка ядра.

 

Во, сделал со Sleep. LoadLibrary и GetProcAddress вызываются только раз. Вся работа с памятью относительная) Плюс убрана 100%-ая загрузка процессора при убирании фокуса с окна. Код

Код:

Base:        //0x41CEA0 - вызывается по 0x4056FA
push ebx
call +5
pop ebx
sub ebx, offset
jmp Wait


LoadSleep:
mov eax, [ebx+SleepAddr]
test eax, eax
jnz .exit

lea eax, [ebx+'KERNEL32.DLL']
push eax
call LoadLibraryA
test eax, eax
jz .exit
mov [ebx+KernelDesc], eax

lea ecx, [ebx+'Sleep']
push ecx
push eax
call GetProcAddress
test eax, eax
jz .close&exit
mov [ebx+SleepAddr], eax
jmp .exit

.close&exit:
mov eax, [ebx+KernelDesc]
push eax
call FreeLibrary

.exit:
ret


LoadTime:
mov eax, [ebx+TimeAddr]
test eax, eax
jnz .exit

lea eax, [ebx+'WINMM32.DLL']
push eax
call LoadLibraryA
test eax, eax
jz .exit
mov [ebx+WinmmDesc], eax

lea ecx, [ebx+'timeBeginPeriod']
push ecx
push eax
call GetProcAddress
test eax, eax
jz .close&exit
mov [ebx+TimeAddr], eax
jmp .exit

.close&exit:
mov eax, [ebx+WinmmDesc]
push eax
call FreeLibrary

.exit:
ret


OurSleep:
mov eax, [ebx+SleepAddr]
test eax, eax
jz .exit

push 1
call eax

.exit:
ret


OurTime:
mov eax, [ebx+TimeWasCall]
test eax, eax
jnz .exit

mov eax, [ebx+TimeAddr]
test eax, eax
jz .exit

push 1
call eax
inc eax
mov [ebx+TimeWasCall], eax

.exit:
ret


Wait:
call LoadSleep
call LoadTime
call OurTime

.cycle:
call GetTickCount
mov edx, [ebx+oldtick]
sub eax, edx
sub eax, 0Eh
jae .exit
call OurSleep
jmp .cycle

.exit:
call GetTickCount
pop ebx
ret


Sleep1:          //0x41CFB1 - вызывается из 0x403F93
call LoadSleep
call OurSleep
jmp 0x404105

 

Offtop

Ну это не так уж страшно. Практически все старые игры грузят процессор под 100%)

 

Немного исправил свой быдлокод, ибо оказалось, что иногда вылетает, поскольку в функции Sleep1 (sub_41CFB1) не рассчитывался базовый адрес. Также пытаюсь исправить фуллскрин по Alt+Enter (изображение превращается в кашу) - пока нормально выглядит только при разрешениях, высота у которых меньше или равна 5/8 ширины . С релоками разберусь попозже, когда сделаю более-менее нормально, отредактирую таблицу.

 

По идее, релоки только для dll-файлов нужны, а для приложений их можно вполне себе убрать:
StripReloc.exe ALADDINW.EXE
Т.к. они нужны в очень редких случаях, когда программа специально задумывалась для работы в двух режимах: и как приложение и как dll-библиотека одновременно.

 

iunnamed, я все-таки хочу оставить, вдруг через 20 лет кто-то будет делать что-то вроде KeeperFX

 

У меня вот что пока получилось: Посмотреть вложение AL95_fix.7z .

• Вставил в таблицу импорта Sleep из kernel32.dll.
• Переделал процедуру таймера 4056F2, чтобы GetTickCount вызывалась только один раз, а вместо второго вставил Sleep(16), таблицу релокаций соответственно поправил. Время задержки можно менять байтом dword'ом 4B23 - т. е. сейчас ~60 к/с. У меня с такой частотой кадров работает вроде бы хорошо.
• Разобрался, почему не создавалось окно - использовался старый класс common controls "msctls_statusbar" (библиотека commctrl.dll), хотя сама игрушка подгружает библиотеку comctl32.dll, а в ней класс называется "msctls_statusbar32".
  В 95-м этот фокус прокатывал, возможно, потому, что старую библиотеку использовала сама система, и она в любом случае была загружена в память, а в XP - уже нет. Поменял название класса и вернул назад всё, связанное со строкой состояния - теперь она нормально создаётся:
  
  

 

У меня медленнее. Особенно заметно, когда прыгаешь с веревки в сторону - гораздо дальше получается

Круто!

 

Вот переписал в твоем файлике sub_4056F2, воткнув туда свой цикл со Sleep. Релоки поправил.

 

Прикольно, теперь с нормальной скоростью даже в развёрнутом режиме работает - добежал до уровня "THE ESCAPE", где успешно просадил все жизни.
Кое-что поправил:
- вернул исправление "ALADDIN.INI" -> ".\SETUP.INI"
- изменил проверку на 256 цветов - игра и так нормально работает (проверено в том числе и в Windows 7 x64) - теперь не будет запускаться только если меньше 256 цветов

 

О, что 256 цветов больше не просит - круто)
Добавил (в твой файл) в процедуру обработки сообщений код, который вызывает Sleep, если окно не в фокусе, чтобы ядро не грузилось на 100%, когда окно неактивно. Пытался сделать так раньше, но в результате Sleep вызывался только при потере фокуса один раз. Теперь нормально.

 

iunnamed, а если игра будет запускаться из Program Files не от имени администратора, Виста/7/8+ даст сохранить настройки прямо в папку программы?

Встроил предложенные исправления: Посмотреть вложение AL95_fix.7z .

- По совету iunnamed включил полноцветные режимы, "ALADDIN.INI" пока оставил как есть.
- По совету AxXxB поменял способ ожидания в таймере и вставил Sleep в цикл простоя (убрал лишний cmp, поэтому хватило места прямо там ).
- Немного упростил таймер, "msctls_statusbar32" перенёс в освободившееся место.

Вроде разобрался, почему портится картинка

в режиме "256 цветов" на вкладке "совместимость".
При запуске игры в режиме совместимости сначала устанавливается 8-битный режим экрана и игра настраивает себя на 8 бит в окне, но после включения полного экрана (SetDisplayMode, 404F95, номера методов IDirectDraw есть тут), а потом возврата в окно RestoreDisplayMode (4050B6) восстанавливает не 8-битный режим, а обычный 16/24/32-битный. Но игра не знает, что режим восстановился неправильно, и копирует палитровую картинку на полноцветный экран.
Если экран изначально был 8-битный (не из-за совместимости), он восстанавливается верно, и окно отображается без ошибки.
Поэтому совместимость лучше не включать (она уже не требуется).

 

У кого-нибудь ещё такой глюк наблюдается?

 

Хм, погонял немного - всё-таки ощутимая разница есть (сверху - непатченная версия в Windows 95 на VirtualBox).
Может кто-нибудь проверить на настоящем компе с Windows 95/98/ME, насколько близка скорость без патчей к верхней картинке?
Чтобы было на что ориентироваться.



Копия на postimg.org.

 

а может лучше вообще выкинуть любое упоминание функции Sleep (1) ?

то есть вообще не использовать Sleep,

а что бы задать паузу - использовать busy waiting

- это по сути простейший цикл считающий, например, синусы до тех пор пока пройдёт достаточное время.


вместо GetTickCount есть QPC, но имхо большая точность не нужна и сойдёт GetTickCount.

 

Думаю, это нужно внести в памятку, если ещё нету, что старые игры на новых системах категорически запрещено ставить в "Program Files" / "Program Files (x86)" - там свои ограничения прав действуют, так что ничем хорошим это, как правило, не кончается.
В %SystemRoot% (C:\Windows) тоже права ограничены, но даже если человек запустил игру из-под администратора, то искать, выковыривать и удалять оттуда при удалении игры файл настроек - удовольствие ниже среднего. С учётом того, что достаточно много игр пытаются сохранить туда настройки - свалка там будет ещё та.
А так всё просто: поиграл, удалил, оставил только архив с игрой где-нибудь в игровых дистрибутивах, и ковыряться в системе, удаляя из неё мусор, не нужно.