Баг с модификацией стека

[Ivan_32]

proc lv_item_insert
		mov  ebx,0
	@@:
		push esp
		pushad
		mov  eax,10h
		add  [cAddr],eax
		;invoke _itoa,[cAddr],ADR,16
		popad
		pop  esp
		mov  [lvItem.mask],LVIF_TEXT + LVIF_PARAM
		lea  eax,[ADR]
		mov  [lvItem.pszText],eax
		mov  [lvItem.iSubItem],0
		mov  [lvItem.iItem],ebx
		mov  [lvItem.lParam],ebx
		lea  eax,[lvItem]
	invoke	SendMessage,[lvH],LVM_INSERTITEM,0,eax
		mov  [lvItem.mask],LVIF_TEXT
		inc  [lvItem.iSubItem]
		lea  eax,[DAT]
		mov  [lvItem.pszText],eax
		lea  eax,[lvItem]
	invoke	SendMessage,[lvH],LVM_SETITEM,0,eax
		inc  ebx
		cmp  ebx,20
		jb   @b
		ret
endp

Стоит поставить гденить внутри этого цикла лишний push как программа падает...

                push esp
		pushad
		;Safe Code

                ;Safe Code Ends
		popad
		pop  esp

Чисто теоретически этот блок защищен от модификаций стека. Т.е. напиши я внутри него push - стек останется неизменным. Ну вернее данные останутся но следюущие данные их перезапишут. Но не смотря на это если туда поставить команду скажем push 100h - программа опять же упадет.

Откуда могут расти ноги у этого бага?

[rpy3uH]

довольно-таки непонятный код....
во-первых, достаточно одной pushad вместо push esp, pushad

	lea  eax,[lvItem]
	invoke	SendMessage,[lvH],LVM_INSERTITEM,0,eax
		mov  [lvItem.mask],LVIF_TEXT
		inc  [lvItem.iSubItem]
		lea  eax,[DAT]
		mov  [lvItem.pszText],eax
		lea  eax,[lvItem]
	invoke	SendMessage,[lvH],LVM_SETITEM,0,eax
		inc  ebx ;<--- в EBX ерунда!
		cmp  ebx,20

после вызова API функций большинство регистров изменяется.
А что вообще нужно было?

[Ulex]

Стоит поставить гденить внутри этого цикла лишний push как программа падает...

Код:

push esp
pushad
;Safe Code

;Safe Code Ends
popad
pop esp

Чисто теоретически этот блок защищен от модификаций стека. Т.е. напиши я внутри него push - стек останется неизменным.

Извините, не понимаю... Что значит блок защищен от модификаций стека?
Вызвали push, соответственно esp уменьшился на 4, т.е. вершина стека уже другая? После этого popad естественно всякую ересь восстановит, а не то, что вы в стек pushad_om записали. Т.е. модифицировали указатель стека, будьте любезны вернуть его на место. Или я чего то недопонял?

[Ivan_32]

Это функция записи айтемов в ListView. Нужно положить в DAT и ADR нужные значения и записать их в pszText. Но по какой то причине стоит вызвать функцию itoa или sprintf как программа падает. pushad и push ESP это только для надежности. Но по какой то причине стек всеравно модифицируется. Может тут что с чем то другим проблема?
Да кстати, вот ListView работает только если его создать вручную - через CreateWindowEx. Тот который из ресурсов просто напросто не принимает столбцы.

Ulex, да вы все правильно поняли. Восстанавливается конечно только указатель на стек. Я неправильно выразился.

[Ulex]

А можно всю программу целиком увидеть. Просто есть подозрение, что стек здесь не при чём. Вот к примеру, судя по названию [ADR] - эта переменная должна содержать адрес какой то текстовой строки, однако вот это дальше - lea eax,[ADR], несколько настораживает.

[rpy3uH]

Сообщение от Ivan_32

Но по какой то причине стек всеравно модифицируется. Может тут что с чем то другим проблема?

стек модифицируется API функциями. также эти API функции модифицируют регистры общего назначения. В данном примере в качестве счётчика используется регистр ebx, его надо сохранять перед каждым вызовом API функции и после вызова восстанавливать. Например, так:

push ebx
invoke ...
pop ebx

[Ivan_32]

Вот это работает без каких либо проблем:

proc lv_item_insert
		mov  ebx,0
	@@:
		push esp
		pushad
		mov  eax,10h
		add  [cAddr],eax
		popad
		pop  esp
		mov  [lvItem.mask],LVIF_TEXT ;+ LVIF_PARAM
		lea  eax,[ADR]
		mov  [lvItem.pszText],eax
		mov  [lvItem.iSubItem],0
		mov  [lvItem.iItem],ebx
		;mov  [lvItem.lParam],ebx
		lea  eax,[lvItem]
		push ebx
	invoke	SendMessage,[lvH],LVM_INSERTITEM,0,eax
		pop ebx
		mov  [lvItem.mask],LVIF_TEXT
		inc  [lvItem.iSubItem]
		lea  eax,[DAT]
		mov  [lvItem.pszText],eax
		lea  eax,[lvItem]
		push ebx
	invoke	SendMessage,[lvH],LVM_SETITEM,0,eax
		pop ebx
		inc  ebx
		cmp  ebx,20
		jb   @b
		ret
endp

А вот это вызывает краш:

proc lv_item_insert
		mov  ebx,0
	@@:
		push esp
		pushad
		mov  eax,10h
		add  [cAddr],eax
		invoke _itoa,[cAddr],ADR,16
		popad
		pop  esp
		mov  [lvItem.mask],LVIF_TEXT ;+ LVIF_PARAM
		lea  eax,[ADR]
		mov  [lvItem.pszText],eax
		mov  [lvItem.iSubItem],0
		mov  [lvItem.iItem],ebx
		;mov  [lvItem.lParam],ebx
		lea  eax,[lvItem]
		push ebx
	invoke	SendMessage,[lvH],LVM_INSERTITEM,0,eax
		pop ebx
		mov  [lvItem.mask],LVIF_TEXT
		inc  [lvItem.iSubItem]
		lea  eax,[DAT]
		mov  [lvItem.pszText],eax
		lea  eax,[lvItem]
		push ebx
	invoke	SendMessage,[lvH],LVM_SETITEM,0,eax
		pop ebx
		inc  ebx
		cmp  ebx,20
		jb   @b
		ret
endp

При чем функция _itoa работает где угодно вполне корректно, только в этой процедуре вызывает краш. Это такие особенности записи значений в ListView что ли?

UPDATE:
Нашел решение проблемы:
invoke sprintf,ADR,xtoa,1000
pop eax
pop eax
pop eax

[rpy3uH]

функция sprintf является функцией из библиотеки С++, и вызывается по соглашению cdecl. При использовании формата cdecl стек должен очищать не вызываемый, а вызывающий.