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

Add a few component identifications #987

Merged
merged 5 commits into from
Jan 18, 2024
Merged

Add a few component identifications #987

merged 5 commits into from
Jan 18, 2024

Conversation

jblu42
Copy link
Contributor

@jblu42 jblu42 commented Jan 10, 2024

  • What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
    Add more component identifications

  • What is the current behavior? (You can also link to an open issue here)
    Components are not identified

  • What is the new behavior (if this is a feature change)? If possible add a screenshot.
    Components are identified

  • Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
    No

  • Other information:

Signed-off-by: Jens Lucius <github@jenslucius.de>
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for contributing a pull request!

Welcome to the EMBA firmware analysis community!

We are glad you are here and appreciate your contribution. Please keep in mind our contributing guidelines here and here.
Also, please check existing open issues and consider to open a discussion in the dedicated discussion area.
Additionally, we have collected a lot of details around EMBA, the installation and the usage of EMBA in our Wiki.

If you like EMBA you have the chance to support us by becoming a Sponsor or buying some beer here.

This is an automatic message. Allow for time for the EMBA community to be able to read the pull request and comment on it.

Copy link
Member

@m-1-k-3 m-1-k-3 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for this great update of the version identifiers. Could you please take a look at my comments?

libjpeg;;bsd-style;"Independent\ JPEG\ Group.s\ CJPEG,\ version\ [0-9][a-z]\ \ [0-9]+-[A-Za-z]+-[0-9]+$";"sed -r 's/Independent\ JPEG\ Group.s\ CJPEG,\ version\ ([0-9][a-z])\ .*/libjpeg:\1/'";
libjpeg-turbo;;bsd-style;"^libjpeg-turbo\ version\ [0-9](\.[0-9]+)+?\ \(build\ [0-9]+\)$";"sed -r 's/libjpeg-turbo\ version\ ([0-9](\.[0-9]+)+?)\ .*/libgjpeg-turbo:\1/'";
libjson-c;multi_grep;mit;'"^INTERNAL ERROR: _json_c_visit"&&"^[0-9](\.[0-9]+)?+$"';"sed -r 's/([0-9](\.[0-9]+)+?)$/json-c:\1/'";
liblz4.so;strict;unknown;"^[0-9](\.[0-9]+)+?$";"sed -r 's/([0-9](\.[0-9]+)+?)$/liblz4:\1/'";
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

do we have some kind of identfier that we can use for multi_grep? We are now trying to avoid strict identifiers.
Same on 353 and 355

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If you mean liblz4 here, no unfortunately there was no unique identifier. With strings I only get the function names and these are not unique. I can provide the library to you if you want to check if there is a better idea.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can we search for multiple non unique strings? We can combine more than two search strings

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I know, priority for me would be to avoid false positives, I have no idea what to use then, here is the library:
liblzma.so.5.2.5.zip

Copy link
Member

@m-1-k-3 m-1-k-3 Jan 10, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hmmm, we could use the library SONAME:

 0x000000000000000e (SONAME)             Library soname: [liblzma.so.5]


└─$ strings ~/Downloads/liblzma.so.5.2.5 | grep liblzma
liblzma.so.5

and one function name like "lzma_version_string"
The third identifier ist then the version itself

What do you mean?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@m-1-k-3 : I currently have problems that the multi grep is not working any more for me. I thought i messed up my environment, but even after a clean reinstall it does not work. Can you quickly confirm that multi-grep is still working for you in the latest emba version?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not good ... not good
Will check here

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

indeed ... it is broken. Will fix it

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you do a check if it is also working for you again?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@m-1-k-3 : Working again and I just verified that my additions are working. Do we have open points to fix ?

config/bin_version_strings.cfg Outdated Show resolved Hide resolved
config/bin_version_strings.cfg Outdated Show resolved Hide resolved
config/bin_version_strings.cfg Show resolved Hide resolved
config/bin_version_strings.cfg Outdated Show resolved Hide resolved
…identifiers

Signed-off-by: Jens Lucius <github@jenslucius.de>
@m-1-k-3 m-1-k-3 mentioned this pull request Jan 17, 2024
jblu42 and others added 3 commits January 18, 2024 09:20
Add multi-grep for liblzma and fix libxml

Signed-off-by: Jens Lucius <github@jenslucius.de>
Signed-off-by: Jens Lucius <github@jenslucius.de>
@m-1-k-3
Copy link
Member

m-1-k-3 commented Jan 18, 2024

Thank your for your work with all these new identifiers.
Good job :)

@m-1-k-3 m-1-k-3 merged commit a647ac7 into e-m-b-a:master Jan 18, 2024
13 checks passed
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.

None yet

2 participants