C/C++ Инлайн ASM неподходящий операнд типа

Question

Я следующий код, который должен XOR блок памяти:

void XorBlock(DWORD dwStartAddress, DWORD dwSize, DWORD dwsKey) 
{ 
DWORD dwKey; 
__asm 
{ 
    push eax 
    push ecx 
    mov ecx, dwStartAddress   // Move Start Address to ECX 
    add ecx, dwSize     // Add the size of the function to ECX 
    mov eax, dwStartAddress   // Copy the Start Address to EAX 

    crypt_loop:       // Start of the loop 
     xor byte ptr ds:[eax], dwKey  // XOR The current byte with 0x4D 
     inc eax       // Increment EAX with dwStartAddress++ 
     cmp eax,ecx      // Check if every byte is XORed 
    jl crypt_loop;      // Else jump back to the start label 

    pop ecx // pop ECX from stack 
    pop eax // pop EAX from stack 
} 
} 

Однако аргумент dwKey дает мне ошибку. Код работает отлично, если, например, dwKey заменяется на 0x5D.

Answer 1

Я думаю, у вас есть две проблемы.

Во-первых, «xor» не может принимать два операнда памяти (ds: [eax] - это ячейка памяти, а dwKey - ячейка памяти); во-вторых, вы использовали «байт ptr», чтобы указать, хотите ли вы байт, но вы пытаетесь использовать DWORD, и сборка не может автоматически преобразовать их.

Итак, вам, вероятно, придется загрузить свое значение в 8-битный регистр, а затем сделать это. Например:

void XorBlock(DWORD dwStartAddress, DWORD dwSize, DWORD dwsKey) 
{ 
    DWORD dwKey; 
    __asm 
    { 
     push eax 
     push ecx 
     mov ecx, dwStartAddress   // Move Start Address to ECX 
     add ecx, dwSize     // Add the size of the function to ECX 
     mov eax, dwStartAddress   // Copy the Start Address to EAX 
     mov ebx, dwKey     // <---- LOAD dwKey into EBX 

     crypt_loop :       // Start of the loop 
      xor byte ptr ds : [eax], bl  // XOR The current byte with the low byte of EBX 
      inc eax       // Increment EAX with dwStartAddress++ 
      cmp eax, ecx      // Check if every byte is XORed 
      jl crypt_loop;      // Else jump back to the start label 

     pop ecx // pop ECX from stack 
     pop eax // pop EAX from stack 
    } 
} 

Хотя, он также выглядит dwKey не инициализирован в коде; возможно, вам нужно просто «mov bl, 0x42». Я также не уверен, что вам нужно нажать и поместить регистры; Я не могу вспомнить, какие регистры вам разрешают сжимать с помощью встроенного ассемблера MSVC++.

Но, в конце концов, я думаю, что Алан Стокс прав в своем комментарии: это очень маловероятно, что сборка на самом деле быстрее, чем код C/C++ в этом случае. Компилятор может легко сгенерировать этот код самостоятельно, и вы можете обнаружить, что компилятор действительно делает неожиданные оптимизации, чтобы заставить его работать даже быстрее, чем «очевидная» сборка (например, loop unrolling).