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

[infra] Support RPM-based distributions with SELinux enabled #10352

Open
0x34d opened this issue May 19, 2023 · 6 comments
Open

[infra] Support RPM-based distributions with SELinux enabled #10352

0x34d opened this issue May 19, 2023 · 6 comments

Comments

@0x34d
Copy link
Contributor

0x34d commented May 19, 2023

I was doing some testing of new local build, and trying to fuzz it.

My command:

python infra/helper.py build_image open5gs
python infra/helper.py build_fuzzers open5gs --engine libfuzzer --sanitizer address
python infra/helper.py check_build open5gs --engine libfuzzer --sanitizer address

python infra/helper.py run_fuzzer open5gs --engine libfuzzer --sanitizer address gtp_message_fuzz
ERROR:__main__:gtp_message_fuzz does not seem to exist. Please run build_fuzzers first.

Debug:

ls build/out/open5gs/

gtp_message_fuzz  gtp_message_fuzz_seed_corpus.zip  lib  llvm-symbolizer  nas_message_fuzz  nas_message_fuzz_seed_corpus.zip
@jonathanmetzman
Copy link
Contributor

I can't repro this so not sure what I can do here.

@0x34d
Copy link
Contributor Author

0x34d commented May 23, 2023

I can't repro this so not sure what I can do here.

Just Fedora's problem :)

@0x34d 0x34d closed this as completed May 23, 2023
@jonathanmetzman
Copy link
Contributor

I can't repro this so not sure what I can do here.

Just Fedora's problem :)

Ah sorry, we use Debian stuff. If you can figure out the issue and make a not-very-invasive patch we probably would accept it.

@evverx
Copy link
Contributor

evverx commented May 25, 2023

I think this issue should just be reopened and repurposed to cover RPM-based distributions with SELinux enabled. It could also probably be linked to #4774. podman is much better integrated into those distributions and is usually used instead of docker there.

@0x34d 0x34d changed the title ERROR "Please run build_fuzzers first." [infra] Support RPM-based distributions with SELinux enabled May 26, 2023
@0x34d 0x34d reopened this May 26, 2023
@0x34d 0x34d closed this as completed May 26, 2023
@evverx
Copy link
Contributor

evverx commented May 26, 2023

@0x34d FWIW I think it would make sense to keep the issue open. Even though it's unlikely to be fixed I think it should make it easier for people using RPM-based distributions to figure out whether it's supposed to work or not.

@0x34d
Copy link
Contributor Author

0x34d commented May 26, 2023

Fix: Don't use moby-engine docker use podman-docker

@0x34d 0x34d reopened this May 26, 2023
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 a pull request may close this issue.

3 participants