Из-за чего ошибка чтения?

Question

[Денис Гончаренко]

Код:

unsigned char source[] = { ... };
    int sourceSize = sizeof(source);
    DWORD old_protect;
    VirtualProtect(&source, sourceSize, PAGE_EXECUTE_READWRITE, &old_protect);
    void(*f)() = (void(*)()) &source;
    f();

Проблема:

Comments

[Александр Ананьев]

На какой строчке вылезает Access violation ?

[Денис Гончаренко]

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

[m4dEngi]

А разве source сам по себе не указатель на область памяти с телом функции?
Если так, то здесь

VirtualProtect(&source, sourceSize, PAGE_EXECUTE_READWRITE, &old_protect);

&source указатель на указатель (int?), 4-8 байт, которым мы ставим резрешение на исполнение и вызываем как функцию далее, вместо функции.

Если не прав, поправьте. Кодер я так себе...

[Денис Гончаренко]

m4dEngi, пробовал менять, никакого эффекта, да и ругается он на другую строку.

[Anton Mashletov]

Денис Гончаренко, Кстати да. А разве source сам по себе не указатель? VirtualProtect(source, ... работать не будет?

[Денис Гончаренко]

Anton Mashletov, работает, но ошибка та же, и возникает она на другой строчке, см. комментарий выше.

[Anton Mashletov]

Денис Гончаренко, возможно бред, но может попробовать создать массив байт не в стеке, а динамически выделить память на source?

[Anton Mashletov]

Денис Гончаренко, а может быть что сам код в source выдает эту ошибку?

[Денис Гончаренко]

Anton Mashletov, не, там просто вывод строки для теста

[Денис Гончаренко]

Anton Mashletov, пробую динамически память выделить:

void * executable_area = VirtualAlloc(NULL, sourceSize, MEM_RESERVE, PAGE_EXECUTE_READWRITE);
memcpy(executable_area, source, sourceSize);

тут ошибка такая же только на memcpy

[Anton Mashletov]

Денис Гончаренко, а можете пример дать source. хочу у себя проверить

[Anton Mashletov]

Денис Гончаренко, MEM_COMMIT | MEM_RESERVE пробовали?

[Денис Гончаренко]

Anton Mashletov,
unsigned char source[] = "\xc7\x85\xc4\xf8\xff\xff\x00\x00\x00\x00\xc7\x45\xfc\xff\xff\xff\xff\x8d\x8d\x40\xfb\xff\xff\xe8\x0f\x6c\xff\xff\x8b\x85\xc4\xf8\xff\xff\xeb\x22";

[Денис Гончаренко]

Anton Mashletov, нашел ошибку. Проблема действительно в source.

[Anton Mashletov]

Денис Гончаренко, блин, а уже VS запустил )

[Денис Гончаренко]

Anton Mashletov, можете тогда помочь, чтобы получать нормальный байтовый код из функции))

Answer 1

[Mercury13]

Вы компилируете функцию в двоичный код и на месте же вызываете. Так что есть вопросы.
1. Функции обеспечили правильное соглашение вызова?
Мне что-то кажется, что при подобной ручной компиляции проще работать с соглашением PASCAL или STDCALL — ну, шут его знает.
2. VirtualProtect работает с целыми страницами. Так что на куске памяти из сегмента данных (ну или из стека, полного кода не вижу) она в лучшем случае откажет, в худшем ничего не сделает. Проверьте, была ли ошибка.
3. Первый параметр должен быть source, а не &source.

Answer 2

[Павел Каптур]

В добавлении к Mercury13, в предпоследней строке должно быть void(*f)() = (void(*)()) source;
А вообще, лучше выделить отдельную страницу, скопировать туда shell, и потом вызывать