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

Looks like openssl is still linked to bulk_extractor? #475

Open
chenrui333 opened this issue Sep 5, 2024 · 3 comments
Open

Looks like openssl is still linked to bulk_extractor? #475

chenrui333 opened this issue Sep 5, 2024 · 3 comments

Comments

@chenrui333
Copy link

chenrui333 commented Sep 5, 2024

👋 I can still see openssl linkage for bulk_extractor for the homebrew build, but in the build note, it said openssl is not required, is that true?

# Note: openssl no longer required

Also another thing, it looks like abseil is linked for macos builds, but not linux build, did I miss any build config? Thanks!

relates to Homebrew/homebrew-core#183451

@simsong
Copy link
Owner

simsong commented Sep 5, 2024

Hi. Sorry about the confusion here. Let me try to clarify:

  • bulk_extractor requires OpenSSL on some platforms, but not on MacOS, as it uses the OS-provided encryption routines there.
  • bulk_extractor now really wants to use re2 which requires abseil. So it really does require both!
  • This is a problem for the mingw compilation, since I can't find a decent re2 and abseil package for mingw.
  • I also can't find another decent regex package that will handle this regex gracefully: .*@company.com.

Please let me know if you have suggestions to help me address above.

@chenrui333
Copy link
Author

  • bulk_extractor requires OpenSSL on some platforms, but not on MacOS, as it uses the OS-provided encryption routines there.

but if openssl present, then it would link to the openssl instead of os-provided crypto?

  • bulk_extractor now really wants to use re2 which requires abseil. So it really does require both!

I thought about it, make sense to me, but somehow on the brew linux build, the abseil did not get linked, do you know why?

@simsong
Copy link
Owner

simsong commented Sep 5, 2024

  • bulk_extractor requires OpenSSL on some platforms, but not on MacOS, as it uses the OS-provided encryption routines there.

but if openssl present, then it would link to the openssl instead of os-provided crypto?

No. bulk_extractor uses dfxml for its hashing, and if common crypto is provided, it will use that instead of openSSL:

https://github.com/dfxml-working-group/dfxml_cpp/blob/1e92fe8ac74e0319324ad3956ce99b54455258fa/src/hash_t.h#L55

  • bulk_extractor now really wants to use re2 which requires abseil. So it really does require both!

I thought about it, make sense to me, but somehow on the brew linux build, the abseil did not get linked, do you know why?

Perhaps it's in the base library somewhere.

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

No branches or pull requests

3 participants
@chenrui333 @simsong and others