Доброго дня. Помогите разобраться, пожалуйста.

Копаю программу, в которой есть такая функция (как ее видит OllyDBG):



Мне нужно сделать так, чтобы она всегда возвращала значение 3. К сожалению, с патчингом бинарников, в отличие от .NET, я не очень знаком. Попробовал сделать так:



После такого "патчинга" программа крэшится. Если открыть исходный exe в IDA и проделать те же изменения через Edit -> Patch Program, IDA начинает ругаться "endp ; sp-analysis failed".

Я понимаю что делаю какую-то очень банальную ошибку, но пока не хватает познаний разобраться, что не так. Подскажите, пожалуйста, как правильно изменить этот код, чтобы получать на выходе нужное мне значение.

Спасибо.

| Сообщение посчитали полезным:

Ты стек невыравненным оставил, retn прыгает по адресу на макушке стека, а там что попало лежит. Если не критично вызывать client.0045EC50, воткни mov eax,3/retn в самое начало функции (0045EC10)

-----
2 оттенка серого

| Сообщение посчитали полезным:

О. Получилось. Спасибо!

| Сообщение посчитали полезным:

как же смешно смотреть когда пишут

когда можно красиво сделать


-----
Чтобы правильно задать вопрос, нужно знать большую часть ответа. Р.Шекли.

| Сообщение посчитали полезным: plutos

ClockMan

Это тоже не красиво. А если нужно куда длинее макро вставить ?
Нужно переместить всю процедуру, так как в ней могут быть релатив ветвления/циклы. Что вы в таком случае сделаете ?

Добавлено спустя 19 минут
Пример.



cfe1_31.12.2018_EXELAB.rU.tgz - move.7z

-----
vx

| Сообщение посчитали полезным:

difexacaw
Изменять исполняемый код - это тоже некрасиво, нужно декомпилировать приложение в исходный код, на котором оно написано, внести нужные изменения, скомпилировать и получить что требуется.

-----
Everything is relative...

| Сообщение посчитали полезным:

Vamit

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

Пересборка кода покрывает все задачи такого рода. Штатно бинарный код описывается графом, изменяется и билдится. Это всегда замечательно работало, только так выполняется любой патч.

-----
vx

| Сообщение посчитали полезным:

Vamit пишет:
Изменять исполняемый код - это тоже некрасиво, нужно декомпилировать приложение в исходный код, на котором оно написано, внести нужные изменения, скомпилировать и получить что требуется
Конечно когда мне нужно взломать программу я сначало её декомпелирую а потом собираю обратно очень удобно кстати

-----
Чтобы правильно задать вопрос, нужно знать большую часть ответа. Р.Шекли.

| Сообщение посчитали полезным:

Vamit пишет:
нужно декомпилировать приложение в исходный код

декопилировать тоже не красиво, нужно написать новое приложение с требуемым функционалом

| Сообщение посчитали полезным:

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

к примеру
хочется поиграть в фифа
игра стоит денег
денег нет
ломать долго и дорого
решение, пройтись по спорт площадкам где играют в футбол и присоединится к игре

| Сообщение посчитали полезным: plutos, ClockMan

выйти на улицу? Слишком экстремально.

| Сообщение посчитали полезным: morgot

Gideon Vi пишет:
выйти на улицу? Слишком экстремально.

ничего экстремального,
особенно если под рукой есть экземпляр книги "Увлекательная травматология"

-----
Give me a HANDLE and I will move the Earth.

| Сообщение посчитали полезным:

Gveon, обычно пишут в самом начале, но иногда проге нужно пройти всю процедуру, тогда в конце писать надо и RETN нужно перезаписывать, если место снизу позволяет ее восстановить обратно.
Судя по коду, тебе лучше бы патчить функцию в CALL client.0045EC50, она там явно записывает нужное значение и выводит его в DWORD PTR SS:[ESP+8], и это значение может вызываться из этой ячейки прогой повторно в другом месте. Так что лучше перестраховаться и писать именно в ячейку там, где это должно писаться, а то с виду ты не там патчишь.

-----
Array[Login..Logout] of Life

| Сообщение посчитали полезным:

Kindly

> это значение может вызываться из этой ячейки прогой повторно в другом месте.

Не может, так как сразу после чтения локальной переменной текущим тредом она уничтожается: esp+16.

-----
vx

| Сообщение посчитали полезным: