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

Jump to bottom

Cleanup output of s10 - vul_func_basic_check function #40

Closed
m-1-k-3 opened this issue Jan 22, 2021 · 3 comments · Fixed by #65
Closed

Cleanup output of s10 - vul_func_basic_check function #40

m-1-k-3 opened this issue Jan 22, 2021 · 3 comments · Fixed by #65
Assignees
@p4cx
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed

Comments

@m-1-k-3
Copy link
Member

m-1-k-3 commented Jan 22, 2021

The output of objdump is misaligned:

image
@m-1-k-3 m-1-k-3 added enhancement New feature or request help wanted Extra attention is needed good first issue Good for newcomers labels Jan 22, 2021
@p4cx
Copy link
Member

p4cx commented Mar 10, 2021

How do you then imagine it, for example, when something like that appears?

    00421a30	DF	*UND*	00000000	strcpy
    004219c0	DF	*UND*	00000000	printf
    004218f0	DF	*UND*	00000000	sprintf
    004218e0	DF	*UND*	00000000	vsnprintf
    004218c0	DF	*UND*	00000000	fprintf
    0041d4b4	g	DF	.text 	00000074	Base	lxmldbc_system
    00402d94	g	DF	.text 	00000084	Base	client_printf
    00421710	DF	*UND*	00000000	snprintf
    00421600	DF	*UND*	00000000	strcat
    004215d0	DF	*UND*	00000000	vfprintf
    00421580	DF	*UND*	00000000	system

@m-1-k-3
Copy link
Member Author

m-1-k-3 commented Mar 10, 2021

Currently we get a huge amount of uninteresting stuff. We have to only search for the functions we have in our config files and then the alignment is probably automatically right.

@p4cx
Copy link
Member

p4cx commented Mar 11, 2021

Is a small fix, only added -w to one grep command in S10 - will add this fix to #65

@p4cx p4cx linked a pull request Mar 11, 2021 that will close this issue
Cleanup #65
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@p4cx p4cx

Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Cleanup p4cx/emba
2 participants
@m-1-k-3 @p4cx