Skip to content

Bug fix with post scripts#40

Open
elikaski wants to merge 1 commit intoxoreaxeaxeax:masterfrom
elikaski:master
Open

Bug fix with post scripts#40
elikaski wants to merge 1 commit intoxoreaxeaxeax:masterfrom
elikaski:master

Conversation

@elikaski
Copy link
Copy Markdown

@elikaski elikaski commented Dec 3, 2020

Compiling a simple Hello World program generates assembly code that contains:

	# (external call)
	movl (sp), %esp  # <REQ>
	movl $printf, (external)

When using post/xor.py (and other post scripts), the corresponding instructions generated are:

	# (external call)
	#xor> movl (sp), %esp  # <REQ>
	xorl %ebx, %ebx
	xorl (sp), %ebx
	xorl %esp  # <REQ>, %ebx
	xorl %ebx, %esp  # <REQ>

The third instruction is the problematic:
xorl %esp # <REQ>, %ebx

The root cause is that the comment (# <REQ>) is also being identified as the "destination" register
My change makes it discard the comment

Compiling a simple Hello World program generates assembly code that contains:

	# (external call)
	movl (sp), %esp  # <REQ>
	movl $printf, (external)

When using post/xor.py (and other post scripts), the corresponding instructions generated are:

	# (external call)
	#xor> movl (sp), %esp  # <REQ>
	xorl %ebx, %ebx
	xorl (sp), %ebx
	xorl %esp  # <REQ>, %ebx
	xorl %ebx, %esp  # <REQ>

The third instruction is the problematic:
	xorl %esp  # <REQ>, %ebx

The root cause is that the comment (# <REQ>) is also being identified as the "destination" register
My change makes it discard the comment
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant