Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Issue with plugins options #2

Closed
GoogleCodeExporter opened this issue Jun 19, 2015 · 2 comments
Closed

Issue with plugins options #2

GoogleCodeExporter opened this issue Jun 19, 2015 · 2 comments

Comments

@GoogleCodeExporter
Copy link

Por cierto haciendo los cambios en la versión de 64bits he visto un posible 
error en la de 32bits, concretamente en esta parte

invoke IsDlgButtonChecked, hWin, RealignPECheckBox
.if eax != TRUE
   mov eax, dword ptr[OptionsArray]
   mov dword ptr[eax], 0
.endif 
invoke IsDlgButtonChecked, hWin, CopyOverlayDataBt
.if eax != TRUE
mov eax, dword ptr[OptionsArray+4]
mov dword ptr[eax],0
.endif

Primero de todo que no sería TRUE, sería BST_CHECKED
Pero sobre todo que tu partes de la inicialización del vector a 1. Y cuando 
ejecutas el plugin corriges con 0 según esté desmarcada la opción.

Si no cierro el FUU, cambio las opciones y vuelvo a ejecutar un plugin, los 
cambios no se quedan reflejados. 

En 64bits he cambiado esto por:

invoke IsDlgButtonChecked, hWin, RealignPECheckBox
@IF <<cmp eax,BST_CHECKED>>,EQUAL?  
mov rax, qword ptr[OptionsArray]
mov dword ptr[rax],1
@ELSE
mov rax, qword ptr[OptionsArray]
mov dword ptr[rax],0
@ENDIF
invoke IsDlgButtonChecked, hWin, CopyOverlayDataBt
@IF<<cmp eax,BST_CHECKED>>,EQUAL?
mov rax, qword ptr[OptionsArray+8]
mov dword ptr[rax],1
@ELSE
mov rax, qword ptr[OptionsArray+8]
mov dword ptr[rax],0
@ENDIF

note: found by Guan de Dio.

Original issue reported on code.google.com by nahuelr...@gmail.com on 12 Jul 2010 at 5:08

@GoogleCodeExporter
Copy link
Author

Original comment by nahuelr...@gmail.com on 12 Jul 2010 at 5:08

  • Changed state: New

@GoogleCodeExporter
Copy link
Author

fixed at r69.

Original comment by nahuelr...@gmail.com on 13 Jul 2010 at 6:18

  • Changed state: Fixed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant