Исследование кода и исправление багов Rebel Moon Rising

Рассмотрим пример разбора кода и исправление косяков полной версии игры Rebel Moon Rising. Как это ни странно, но в рип-версии таких недочетов почему-то не наблюдается, другая компиляция-с, однако…
Исходный файл: rmr.exe, SIZE=385 536 Bt, дата 18/04/1997, ENGLISH VERSION
1. Проблема
Если любое количество носимых игроком боеприпасов на максимуме (это 200=C8h, для всех), то лежащие боеприпасы на карте почему-то продолжают браться, а навара то нет, поскольку он обрезается по 200, этого необычного явления не наблюдалось в рип-версии.
Я захотел устранить проблему
a) Декомпиляция в самом простом дизассемблере WinDASM.
б) Поиск текстовых строк в декомпилированном тексте rmr.alf : eax, 000000C8, ebx, 000000C8, ecx, 000000C8,… ax, 00C8, bx, 00C8…, cmp eax, 000000C8, cmp ebx, 000000C8…, cmp cx, 00C8… и т. п. где есть операция сравнения или присвоения по 200 (C8h).
Небольшое отступление: Вообще ни дай Бог, чтобы числа, используемые для вывода различных игровых параметров, были кратны степени 2, обычно они кратны 10, 100, 5, что облегчает поиск. Такие числа особо уникальны в арифм. операциях сравнения, сложения. Именно так я и раскатал Alpha Storm, уцепившись только за число 999 (000003E7h)
в) Проверка полученных результатов (смещения WinDASM и в IDA Pro совпадают)
Код

Код:

1) Первое место (это щит, перед этим местом сложение к щиту 50)
:0041D234 668B08                  mov cx, word ptr [eax]
:0041D237 6681F9C800              cmp cx, 00C8
:0041D23C 7C04                    jl 0041D242
:0041D23E 66B9C800                mov cx, 00C8

2) Второе место
:0041D3E5 668B0C450CEC4500        mov cx, word ptr [2*eax+0045EC0C]
:0041D3ED 8D04450CEC4500          lea eax, dword ptr [2*eax+0045EC0C]
:0041D3F4 6681F9C800              cmp cx, 00C8
:0041D3F9 7C04                    jl 0041D3FF
:0041D3FB 66B9C800                mov cx, 00C8

3) 3-е место
:0041CE96 663DC800                cmp ax, 00C8
:0041CE9A 7D3B                    jge 0041CED7
:0041CE9C F6450704                test [ebp+07], 04
:0041CEA0 7404                    je 0041CEA6
:0041CEA2 66BF0100                mov di, 0001

4) 4-е место
:0041D12D 668B1C45F8EB4500        mov bx, word ptr [2*eax+0045EBF8]
:0041D135 6681FBC800              cmp bx, 00C8 {с него идет прыжок без отображения надписей}
:0041D13A 0F8D9A000000            jnl 0041D1DA
:0041D140 83E80A                  sub eax, 0000000A
:0041D143 83F804                  cmp eax, 00000004
:0041D146 7762                    ja 0041D1AA
:0041D148 FF2485D4D44100          jmp dword ptr [4*eax+0041D4D4]
:0041D14F 8B442414                mov eax, dword ptr [esp+14]
:0041D153 8B4C2410                mov ecx, dword ptr [esp+10]
:0041D157 50                      push eax
:0041D158 51                      push ecx

5) 5-е место
:0040604C 3DC8000000              cmp eax, 000000C8
:00406051 730A                    jnb 0040605D
:00406053 8D411C                  lea eax, dword ptr [ecx+1C]
:00406056 A370194500              mov dword ptr [00451970], eax
:0040605B EB05                    jmp 00406062

6) 6-е место (это суммирование подбора батарейки к щиту)
:0041CEA6 0FBF4D26                movsx ecx, word ptr [ebp+26]
:0041CEAA 0FBFC0                  movsx eax, ax
:0041CEAD 03C8                    add ecx, eax {по ней проверить}
:0041CEAF 81F9C8000000            cmp ecx, 000000C8
:0041CEB5 7C05                    jl 0041CEBC
:0041CEB7 B9C8000000              mov ecx, 000000C8

7) 7-е место
:0041CED7 F6460201                test [esi+02], 01
:0041CEDB 7428                    je 0041CF05
:0041CEDD 66817E3CC800            cmp word ptr [esi+3C], 00C8
:0041CEE3 7D20                    jge 0041CF05
:0041CEE5 A1941A4500              mov eax, dword ptr [00451A94]
:0041CEEA 3BF0                    cmp esi, eax
:0041CEEC 7517                    jne 0041CF05

8) 8-е место
:00417323 33C9                    xor ecx, ecx
:00417325 8A88F4744100            mov cl, byte ptr [eax+004174F4]
:0041732B FF248DDC744100          jmp dword ptr [4*ecx+004174DC]
:00417332 66817E26C800            cmp word ptr [esi+26], 00C8
:00417338 0F8477010000            je 004174B5
:0041733E 66C746262800            mov [esi+26], 0028

9) 9-ое место
:0041D1AA 66035D26                add bx, word ptr [ebp+26]
:0041D1AE 6681FBC800              cmp bx, 00C8
:0041D1B3 7C04                    jl 0041D1B9
:0041D1B5 66BBC800                mov bx, 00C8

Создаем соотвествующую игровую ситуацию и сейвимся. Теперь правим байты.
Вот первый ощутимый результат 4)

Код:

4) вариант 
:0041D135 6681FBC800              cmp bx, 00C8
:0041D135 6681FBC700              cmp bx, 00C7
Для боеприпаса нет эффекта. 
:0041D135 6681FBC800              cmp bx, 00C8
:0041D135 6681FBC900              cmp bx, 00C9
Для боеприпаса появляется надпись, когда его берешь при своих носимых 200 единицах.
Далее изменил вот в этом месте
:0041D1AA 66035D26                add bx, word ptr [ebp+26]
:0041D1AA 662B5D26                sub bx, word ptr [ebp+26]

И боеприпасы перестали прибавляться.
Все, с этого места я начал исследовать код.
г) Исследование кода.
Здесь очень помогли наработки Vladimir 777 по исследованию формата сейвов и карт игры.
Помучавшись немного в HEX-редакторе я интуитивно уже изменил байт с 1 на 0, в конце изучаемой процедуры. Поскольку сначала нужно вывести надпись, а затем уже после взятия предмета уничтожить объект на карте, так должно быть по нормальной логике.
Проблема решена

Код:

:0041D1DA F6460201                test [esi+02], 01
:0041D1DE 0F8499020000            je 0041D47D
:0041D1E4 66BF0100                mov di, 0001
:0041D1E8 5D                      pop ebp
:0041D1E9 0FBFC7                  movsx eax, di
:0041D1EC 5F                      pop edi
:0041D1ED 5E                      pop esi
:0041D1EE 5B                      pop ebx
:0041D1EF 83C408                  add esp, 00000008
:0041D1F2 C3                      ret
Меняем
:0041D1E4 66BF0000                mov di, 0000

Проверил – все работает.
Просто постоянно было удаление объекта, независимо от того нужен он или нет в данный момент.

2. Проблема
Гораздо интереснее, из разряда программистского ляпа. Когда берется батарейка к щиту и щит становится равен 200, то надпись о том, что взята батарейка не отображается. Это же и относится к супербатарее, поскольку она всегда дополняет щит до 200.

Код уже частично понят. Дальше легче. Полностью быстро нашел место прибавления к щиту

Код:

:0041CEAD 03C8                    add ecx, eax
меняем
:0041CEAD 2BC8                    sub ecx, eax
Щит стал убавляться при подборе батарейки

Вся процедура подбора довольно небольшая

Код:

:0041CE88 8B742420                mov esi, dword ptr [esp+20]
:0041CE8C F6460201                test [esi+02], 01
:0041CE90 7473                    je 0041CF05
:0041CE92 668B463C                mov ax, word ptr [esi+3C]
:0041CE96 663DC800                cmp ax, 00C8
:0041CE9A 7D3B                    jge 0041CED7
:0041CE9C F6450704                test [ebp+07], 04
:0041CEA0 7404                    je 0041CEA6
:0041CEA2 66BF0100                mov di, 0001

* Referenced by a (U)nconditional or (C)onditional Jump at Address:
|:0041CEA0(C)
|
:0041CEA6 0FBF4D26                movsx ecx, word ptr [ebp+26]
:0041CEAA 0FBFC0                  movsx eax, ax
:0041CEAD 03C8                    add ecx, eax
:0041CEAF 81F9C8000000            cmp ecx, 000000C8
:0041CEB5 7C05                    jl 0041CEBC
:0041CEB7 B9C8000000              mov ecx, 000000C8

* Referenced by a (U)nconditional or (C)onditional Jump at Address:
|:0041CEB5(C)
|
:0041CEBC 66894E3C                mov word ptr [esi+3C], cx
:0041CEC0 0FBFC1                  movsx eax, cx
:0041CEC3 0FBF4E2C                movsx ecx, word ptr [esi+2C]
:0041CEC7 C1E102                  shl ecx, 02
:0041CECA 8D14C9                  lea edx, dword ptr [ecx+8*ecx]
:0041CECD 8D1C52                  lea ebx, dword ptr [edx+2*edx]
:0041CED0 89849BF4D14700          mov dword ptr [ebx+4*ebx+0047D1F4], eax

* Referenced by a (U)nconditional or (C)onditional Jump at Address:
|:0041CE9A(C)
|
:0041CED7 F6460201                test [esi+02], 01
:0041CEDB 7428                    je 0041CF05
:0041CEDD 66817E3CC800            cmp word ptr [esi+3C], 00C8
:0041CEE3 7D20                    jge 0041CF05
:0041CEE5 A1941A4500              mov eax, dword ptr [00451A94]
:0041CEEA 3BF0                    cmp esi, eax
:0041CEEC 7517                    jne 0041CF05
:0041CEEE 8B442414                mov eax, dword ptr [esp+14]
:0041CEF2 8B4C2410                mov ecx, dword ptr [esp+10]
:0041CEF6 50                      push eax
:0041CEF7 51                      push ecx

* Possible StringData Ref from Data Obj ->"Shield"
                                  |
:0041CEF8 68A03C4500              push 00453CA0
:0041CEFD E81E7E0000              call 00424D20
:0041CF02 83C40C                  add esp, 0000000C

* Referenced by a (U)nconditional or (C)onditional Jump at Addresses:
|:0041CE90(C), :0041CEDB(C), :0041CEE3(C), :0041CEEC(C)
|
:0041CF05 668B4506                mov ax, word ptr [ebp+06]
:0041CF09 5D                      pop ebp
:0041CF0A 66257000                and ax, 0070
:0041CF0E 66094602                or word ptr [esi+02], ax
:0041CF12 0FBFC7                  movsx eax, di
:0041CF15 5F                      pop edi
:0041CF16 5E                      pop esi
:0041CF17 5B                      pop ebx
:0041CF18 83C408                  add esp, 00000008
:0041CF1B C3                      ret

Теперь комментарии

Код:

:0041CE88 8B742420                mov esi, dword ptr [esp+20] кладем смещение начала структуры информации об игроке
:0041CE8C F6460201                test [esi+02], 01 че-то у него там проверяем (живой не живой может?) 
:0041CE90 7473                    je 0041CF05 если че-то у него там равно 1 то прыгаем в конец 0041CF05 процедуры
:0041CE92 668B463C                mov ax, word ptr [esi+3C] загружаем сколько сейчас щита
:0041CE96 663DC800                cmp ax, 00C8 сравниваем с 200
:0041CE9A 7D3B                    jge 0041CED7 если равно или больше то прыгаем как раз перед тем переходом который проверял необходимость вывода надписи
:0041CE9C F6450704                test [ebp+07], 04 самое интересное это именно байт 07 структуры объекта на карте из сейва и его проверяют на 4
:0041CEA0 7404                    je 0041CEA6 если равен 4 то перепрыгиваем через строку (не убираем объект с карты, за это и отвечает байт 7 в структуре записи объекта на карте)
:0041CEA2 66BF0100                mov di, 0001 вроде это должно уничтожать потом объект на карте, я как раз такой же операнд  патчил в первой проблеме игры. И ниже edi нигде не задействован (проверил да это действительно уничтожение объекта).
:0041CEA6 0FBF4D26                movsx ecx, word ptr [ebp+26] это байт 26 в структуре записи объекта на карте, сколько чего в объекте витаминов и патронов
:0041CEAA - :0041CEB7 ну тут все понятно.

Все, проверил, решена проблема.
Прыгаем с :0041CE9A ниже сразу на 0041CF05 место начала выхода из процедуры
ЭТО абсолютно безопасное место. А переход :0041CEE3 нопим, теперь при 200 тоже должно название темы отображаться, но мы сюда при 200 уже никогда не попадем, только когда до подбора было менее 200.
Вот так и исправил эту программистскую ошибку.

ВОТ ВКРАТЦЕ РЕЗУЛЬТАТЫ
1. Проблема берущихся боеприпасов при подборе

Код:

   :0041D1E4 66BF0100                mov di, 0001
   :0041D1E4 66BF0000                mov di, 0000

2. Проблема не отображающейся надписи батарейки при подборе

Код:

   :0041CE9A 7D3B                    jge 0041CED7
   :0041CE9A 7D69                    jge 0041CF05
И еще 
   :0041CEE3 7D20                    jge 0041CF05
   :0041CEE3 9090                    nops

А физические смещения в rmr.exe найдите сами по уникальным цепочкам, если хотите.
Вот правленый rmr.exe
http://ifolder.ru/22169355

 

kreol, я отвечу.

Пока это неизвестно, но ничего другого я не замечал. Первый баг замечен мной, второй нашёл WERTA. После перевода Rebel Moon я планирую параллельное прохождение рипа и CD-версии Rebel Moon Rising именно для выяснения отличий и поиска багов. Пока из отличий замечено:
В CD-версии 21 уровень, в рипе 17.
В CD-версии есть возможность сетевой игры, два сетевых персонажа, некоторые объекты для сетевой игры и 21 сетевой уровень.
Одинаковые карты в CD-версии и рипе немного различаются. Вот эти отличия я и хотел выявить в первую очередь. Вдруг на каком-либо уровне они столь велики, что можно будет говорить об отдельной карте.

Я сначала эту идею и пропагандировал. Думал перенести в рип дополнительные четыре уровня и всё. Но не хочется уничтожать возможность сетевой игры. Да и найденные баги WERTA уже поправил.

 

Rebel Moon Rising patcher by WERTA

Вот сделал патч. Применим к любой языковой версии, поскольку оная один и тот же файл. По F3 проверьте при отключенной панели, они даже не удосужились в национальных версиях игры перевести SHIELD, AMMO...торопились?
Проверяйте патч.(на полной версии конечно rmr.exe)

 

Там вообще странненько как-то. CD-версия даже с другой скоростью работает, не говоря уже о том, что у неё другое количество врагов (это отражено как минимум в первом брифинге) и настройки вроде бы чутка отличаются.

 

У них карты различаются немного. В настройках по умолчанию стрейф на Alt, а бег на Shift (в рипе наоборот). Я тоже не пойму как коммерческий продукт вышел с такими ляпами в коде, а рип был без них. Рип явно собран из демки игры. А меньше уровней, так наверное по причине несоместимости с демкой некоторые пришлось выкинуть, это моя версия.

 

Основные различия между версиями 1.0 и 1.10 Rebel Moon Rising.

Поскольку рипы на сайте планируется постепенно заменять на полные версии я провёл сравнение версий 1.0 (рип на сайте, 17 уровней) и 1.10 (раздача, автор раздачи DeaDFausT).

Преимущества 1.10

В ней 21 уровень против 17 в 1.0. В 1.10 добавлены возможности голосового управления (на мой взгляд, бесполезнейшая вещь) и сетевой игры (21 сетевой уровень). Добавлена возможность управления панелью информации в нижней части экрана с помощью последовательных нажатий клавиши F3 (три состояния: панель, только текст, вообще ничего). Добавлена возможность управления яркостью изображения клавишами F11 (затемняет) и F12 (освещает). На мой взгляд – вредная функция. Если где-то в игре должна быть темнота – надо искать очки ночного видения, а не играться с кнопками. Добавлена возможность самоубийства персонажа нажатием клавиши F10. Загадочная функция, предназначенная, скорее всего, для сетевой игры. Увеличена скорость полёта вражеских выстрелов на самой низкой сложности (может и на средней с высокой, но там разница не так заметна).

Недостатки 1.10

Убрана сама концепция дверей и ключей трёх цветов. Их место занято двумя флагами для сетевой игры и «компьютером», похожим на осциллограф. Он применён на уровне №19, нечто типа квестового предмета.
Допущена ошибка в файле плазменной винтовки WPN5.INI. Урон уменьшен с 75 до 45, тем самым, превратив и без того малоэффективное оружие в хлопушку, даром жрущую патроны. Это именно ошибка, а не намеренное изменение, так как и в новой версии файла справки указанный урон - 75.
Кроме того имелись два бага: возможность подбирать патроны с полным боекомплектом (по сути, просто выброс их с карты) и отсутствие надписи о взятии батареи при сопутствующей зарядке щита до полной мощности. Это уже исправил WERTA.

Просто отличия.

В настройках по умолчанию стрейф на Alt, а бег на Shift (в 1.0 наоборот). В финальной картинке 1.10 появился текст «The end» и сообщение, что Rebel Moon Revolution должна появиться к Рождеству 1998 года. Учитывая, что 1.0 представлена на сайте в виде рипа, музыки в ней нет.

Сравнительная таблица

Карта пятого уровня 1.0

Карта шестого уровня 1.10

Карты уровней с минимальными отличиями прикреплять не стал, там не сразу и видно, какое помещение из квадратного стало восьмиугольным и т. д. Мелочи там сплошные.

Кроме всех перечисленных в таблице недочётов в обеих версиях встречаются уровни, на которых возможно прекращение потребления кислорода без взятия кислородного регенератора, просто по наступлению следующей фазы (прохождение каждого уровня может быть разбито на три фазы, потребление кислорода выставляется отдельно в каждой фазе).

Руководствуясь вышеизложенной информацией предлагаю решить стоит ли сохранять на сайте рип (1.0) или заменить его полной версией (1.10). Лично мне хотелось бы как-то сохранить пятый уровень рипа.
Ещё надо решить, стоит ли устранять замеченные недостатки карт и разбираться с расходом кислорода на тех картах, где замечены проблемы.

 

Вообще классно все написано.
Vladimir 777 если ты скриншотишь из игры карту, я могу сделать алгоритм прорисовки ребер, все равно в редакторе это надо будет делать, чтоб не тратить время. Я помню, совершил давно подвиг - заскринил всю карту 14 уровня (Незавершенные субэстроны).
Добавлю в версиях 1.0 и 1.06 ехе-шник один и тот же. Этот ехе-шник спас Rebel Moon (ключи еслиб не оставили...), и на нем будут основаны все последующие моды. Рип версию скрепя сердце надо убирать. Хотя она качественней. Да и путаница начинается с нумерацией уровней.
С косяками разберемся по мере прохождения. Если что, у нас под рукой рип, под него все косячные параметры карты можно настроить. Я удивляюсь, че они все так коряво сделали то. Вроде добавлено мало чего, а код и карты пострадали.

 

С этим я уже закончил, собственно кроме приложенных только три уровня скриншотил. Так что алгоритм отрисовки можешь делать и потом.

Я пришёл к тому же выводу. Из рипа жалко только пятый уровень. Впрочем, его можно добавить к планируемому моду и, таким образом, сохранить. Только, наверное, цели надо будет заменить. И в Rebel Moon и в Rebel Moon Rising они были. Ещё раз расстреливать те же машинки что-то не хочется.

А чего ждать-то? Я уже прошёл, могу начинать делать правки. Плохо, что для устранения возможности прекращение потребления кислорода без взятия кислородного регенератора придётся городить "отсебятину" (менять стартовый запас кислорода, перетаскивать регенератор в другое место карты, имплантировать на карту двери и т. д.). Все остальные проблемы легко устранимы.

Кстати, в виде чего мы будем выдавать правки: набор файлов с заменой оригинальных, патчер, изменения самого образа?

Похоже, как обычно не успевали. Это ж прямо визитная карточка Fenris Wolf - опаздывать со сроками сдачи продукта. Кстати, основная масса косяков находится на тех картах, что появились в только в 1.10. Насчёт кода - не знаю, возможность подбирать боеприпасы с полным боекомплектом могла быть внесена намеренно, в расчёте на сетевую игру. Чтоб врагу патроны не достались. Но теперь актуальнее сингл, а там эта "инновация" очень неудобна.

 

Образ лучше оставить родной. Для аутентичности. Просто выложить к нему неофициальный патч со списком изменений.

 

Это ты про Ammo Pack, так он действительно работает и в 1.0, а в предпоследнем 26 уровне RM он не работал по причине спешки, он в аутеничной ДОС-игре отображается, но не берется. А из под RMR 1.0 отображался как боеприпас инопланетного оружия. В общем это объект в OLS записи которого BYTE#2=01h, BYTE#24=0Fh. Этот Ammo Pack действительно увеличивает все боеприпасы до 200, кроме боеприпасов для инопланетного оружия (что в общем логично).

 

WERTA, да нет, не про Ammo Pack. Я вчера подумал, что исправленный тобой баг (возможность подбирать боеприпасы, когда их запас равен 200) может быть не багом, а предназначенной для сетевой игры возможностью. Т. е. боеприпасов у нас по максимуму, но, чтоб они не доставались врагу, мы их "подбираем" (убираем с карты).
Но эта мысль абсолютно неправильна, так как если бы это не являлось багом, подобная возможность была бы реализована и для батарей щита. И, вообще, это был бы верх нелогичности - совершенно неясно куда и как боец убирает с карты эти боеприпасы.

 

К своему стыду совершенно не знаю специфики сетевой игры в 3D шутерах. Если подобные фишки реализованы в сетевых режимах других игр, то почему бы и нет, просто убираем и все. Нужно подтверждение этой фишки из других шутерных игр.

 

Да нет там этой "фишки", WERTA, это у меня приступ "неудержимого полёта мысли" был. Если бы изменение было намеренно внесено разработчиками, то то же самое было бы сделано для батарей щита. А этого нет.
Так что, не беспокойся, твоя правка совершенно верная, это был баг. Не надо ничего убирать.

 

Сделал патч для Rebel Moon Rising 1.10. На уровнях, где расхода кислорода не было, теперь есть. Сам по себе (изменение фазы игры) расход кислорода больше нигде не отключается. Ну, и мелочи всякие, типа заниженного урона оружия №7 и бесконечно берущихся предметов поправлены. Патч включает в себя правки ЕХЕ, которые ранее сделал WERTA.
список изменений

Rmr.exe

1)убран баг с возможностью подбирать боеприпасы при их максимальном запасе у игрока (это сделал WERTA)
2)убран баг с отсутствием надписи о взятой батарейке при достижении полной зарядки щита(это сделал WERTA)
3)убран баг с изменением надписи «Jetpack» на «Shoot» путём добавления значения «Assign»
4)заголовок подменю соединения переименован из «Multiplayer» в «Connect»

KEYS.DAT

«L ALT» переименован в «ALT», так как игра не различает левую и правую клавиши «ALT».

Rmfont1a.tga

Символы «%», «>» и «'» изменены на пиктограммы, изображающие мышь с нажатыми клавишами. Это необходимо для устранения проблемы с не вмещающимися в подменю AssignView надписями при настройке мыши.

MENUS.INI

Внесены изменения с целью устранения проблемы с не вмещающимися в подменю «AssignView» надписями при настройке мыши.

Папка WEAPONS, WPN5.INI

Исправлен заниженный урон оружия Plasma Cannon (с 45 до 75). Значение 75 выставлено согласно таблице из RMR Help.hlp, а также, по аналогии с более ранними версиями игры.

Папка SCRNS

lvl2c.tga - перенесён крестик, обозначающий местонахождение кратера Фуко (Foucault). Создатели картинки промазали даже с полушарием.
Y.TGA – заменил изображение кислородного регенератора (oxygen recycler) на более соответствующее его новому местонахождению.
LVL3C.TGA - перенесён крестик, обозначающий местонахождение кратера Эйхштедт (Eichstadt). Авторы опять несколько промахнулись.

Папка LEVELS

Содержит необходимые изменения, связанные с устранением проблемы прекращения потребления кислорода при смене фазы игры без взятия кислородного регенератора (oxygen recycler). Изменения, в основном, касаются местоположения кислородного регенератора (oxygen recycler) на картах и строки «Oxygen=0» в соответствующих разделах файлов LVL*.INI. Кроме того, исправлены мелкие баги, связанные с дверями и предметами.

В архиве папка PatchRMR1.10. Содержимое папки надо скопировать в папку с Rebel Moon Rising, заменяя оригинальные файлы.