Проблемы с реализации strstr в NASM

Question

Я боролся с реализацией следующей strstr функции в NASM:

char *strstr(const char *s1, const char *s2) { 
    size_t n = strlen(s2); 
    while(*s1) 
     if(!memcmp(s1++,s2,n)) 
      return s1-1; 
    return 0; 
} 

До сих пор у меня есть следующий код:

global _mystrstr 
extern _strlen 
extern _memcmp 
_mystrstr: 

    ; prologue goes here 

    ; moving s1 to edi and s2 to esi  
    ; pushing n onto the stack and jumping to the loop 

    ; while(*s1) 
    .while_loop: 
     cmp  byte[edi], 0 
     je  .return_null 
     ; memcmp(s1++,s2,n) 
     push dword[esp + 4] 
     push esi 
     push edi 
     call _memcmp 
     add  esp, 12 
     inc  edi 
     ; if(!memcmp(s1++,s2,n)) 
     cmp  eax, 0 
     jne  .while_loop 
     jmp  .return_value 

    .return_value: 
     ; blah blah 

    .return_null: 
     ; standard stuff goes here 

По какой-то причине memcmp никогда не возвращает 0. Я проверил его с printf, распечатав значения, которые имеет eax, и всегда 1 или -1. Может ли кто-нибудь указать, что я могу здесь делать неправильно?

Answer 1

После нажатия eax на стек, esp указывает на это место памяти. Линия

push dword[esp + 4] 

не будет push значение n но значение толкнул заранее.
Заменить строку с

push dword[esp] 

и повторите попытку.