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

app-forensics/afl: Fix type specifier missing, defaults to int #32146

Closed
wants to merge 1 commit into from

Conversation

listout
Copy link
Contributor

@listout listout commented Aug 2, 2023

@listout
Copy link
Contributor Author

listout commented Aug 2, 2023

Please note, that I got this while doing ebuild afl-2.57b-r3.ebuild clean compile

set -e; for i in afl-g++ afl-clang afl-clang++; do ln -sf afl-gcc $i; done
ln -sf afl-as as
[*] Testing the CC wrapper and instrumentation output...
unset AFL_USE_ASAN AFL_USE_MSAN; AFL_QUIET=1 AFL_INST_RATIO=100 AFL_PATH=. ./afl-clang -O2 -pipe -Wall -D_FORTIFY_SOURCE=2 -g -Wno-pointer-sign -DAFL_PATH=\"/usr/lib/afl\" -DDOC_PATH=\"/usr/share/doc/afl-2.57b-r3\" -DBIN_PATH=\"/usr/bin\" test-instr.c -o test-instr -Wl,--as-needed -ldl

[-] PROGRAM ABORT : Incorrect use (not called through afl-gcc?)
         Location : edit_params(), afl-as.c:197

clang-16: error: assembler command failed with exit code 1 (use -v to see invocation)

@gentoo-bot
Copy link

Pull Request assignment

Submitter: @listout
Areas affected: ebuilds
Packages affected: app-forensics/afl

app-forensics/afl: @hannob

Linked bugs

Bugs linked: 894514


In order to force reassignment and/or bug reference scan, please append [please reassign] to the pull request title.

Docs: Code of ConductCopyright policy (expl.) ● DevmanualGitHub PRsProxy-maint guide

@gentoo-bot gentoo-bot added assigned PR successfully assigned to the package maintainer(s). bug linked Bug/Closes found in footer, and cross-linked with the PR. labels Aug 2, 2023
@hannob
Copy link
Member

hannob commented Aug 2, 2023

It appears this is only a check in the Makefile, so we don't need a version bump. Other than that, patch should be ok, will have a look soon and probably commit it.

@listout
Copy link
Contributor Author

listout commented Aug 2, 2023

Fixing (reverting the version bump)

Closes: https://bugs.gentoo.org/894514
Signed-off-by: Brahmajit Das <brahmajit.xyz@gmail.com>
@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2023-08-02 17:30 UTC
Newest commit scanned: e2d8a43
Status: ✅ good

There are existing issues already. Please look into the report to make sure none of them affect the packages in question:
https://qa-reports.gentoo.org/output/gentoo-ci/1162826804/output.html

@gentoo-bot gentoo-bot closed this in 5713308 Aug 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
assigned PR successfully assigned to the package maintainer(s). bug linked Bug/Closes found in footer, and cross-linked with the PR.
Projects
None yet
4 participants