Threaded Mode | Linear Mode

Sav · 06.07.2012 18:55

Я всё-таки разобрался со шрифтами.
Вот патч:

PHP Code:

// Исправляем обработку отрицательного отступа шрифта (ошибку приведения типов).
_PI->WriteByte(0x4B5349 + 1, 0xB6); // (mov)zx
_PI->WriteByte(0x4B53E5 + 1, 0xB6); // (mov)zx 

Ошибка эта была не в алгоритме, а в приведении типов, поэтому её не видно в выложенном мной декомпилированном коде.

Вот код, содержащий ошибку:

Code:

.text:004B5335 ; 109:             if ( *((_DWORD *)&this_v11->v_table + 3 * (symbol_v21 + 5)) < 0 )// fnt->char_sizes[symbol].width

.text:004B5335                 mov     edx, eax

.text:004B5337                 and     edx, 0FFh

.text:004B533D                 add     edx, 5

.text:004B5340                 lea     edx, [edx+edx*2]

.text:004B5343                 cmp     dword ptr [ebx+edx*4], 0

.text:004B5347

.text:004B5347 loc_4B5347:                             ; DATA XREF: 0138630Ao

.text:004B5347                 jge     short loc_4B5357

.text:004B5349 ; 110:               symbols_width_v19 = -*((_DWORD *)&this_v11->v_table + 3 * ((char)symbol_v21 + 5));// fnt->char_sizes[symbol].width

.text:004B5349                 movsx   ecx, al

.text:004B534C

.text:004B534C loc_4B534C:                             ; DATA XREF: 018ECE8Co

.text:004B534C                 add     ecx, 5

.text:004B534F                 lea     ecx, [ecx+ecx*2]

.text:004B5352                 mov     ecx, [ebx+ecx*4]

.text:004B5355                 neg     ecx

Тут левый отступ первого символа строки сравнивается с 0, а потом, если он отрицательный, ecx устанавливается в -него (чтобы потом, когда он в цикле прибавится, скомпенсировать его, чтобы таким образом игнорировать отрицательный отступ первого символа строки).
Проблема в строке ".text:004B5349 movsx ecx, al". Ведь если символ русский, al будет отрицательной и в ecx запишется отрицательное число, которое потом будет использоваться как индекс. В результате, вместо отступа будет взят мусор (например, 0xBAADF00D), из-за которого строка уползёт чёрт знает куда. Ну или там может лежать 0 и недочёт не будет заметен.
Такая же вещь с правым отступом последнего символа строки.

Патч правит movsx на movzx, в результате чего знак игнорируется и для русских букв генерируются нормальные индексы.

Threaded Mode \| Linear Mode Решение багов ERA/WOG » Примеры решения багов ERA/WOG на уровне кода.
Author	Message