Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 21 Nov 2014 18:01:08 +0100
From:      Fabian Keil <freebsd-listen@fabiankeil.de>
To:        freebsd-ports@freebsd.org
Subject:   Re: Request for (i386) testing: american fuzzy lop
Message-ID:  <6f44289d.63c6d622@fabiankeil.de>
In-Reply-To: <fvdc-7g1r-wny@vfemail.net>
References:  <3dc1c153.7b7b9177@fabiankeil.de> <546DF8A5.3060601@gmail.com> <3fb914c3.1002708a@fabiankeil.de> <a93l-6cj5-wny@vfemail.net> <1557502e.1751489a@fabiankeil.de> <fvdc-7g1r-wny@vfemail.net>

next in thread | previous in thread | raw e-mail | index | archive | help
--Sig_/L_6/tikl2bRELzBDQ_6m8yG
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: quoted-printable

Jan Beich <jbeich@vfemail.net> wrote:

> Fabian Keil <freebsd-listen@fabiankeil.de> writes:
>=20
> >>   [*] Testing the CC wrapper and instrumentation output...
> >>   AFL_QUIET=3D1 AFL_INST_RATIO=3D100 AFL_PATH=3D. ./afl-clang -O2 -pipe
> >> -fstack-protector -fno-strict-aliasing -Wall -D_FORTIFY_SOURCE=3D2
> >> -Wno-pointer-sign -DAFL_PATH=3D\"/prefix/afl-0.60b/lib/afl\"
> >> -DDOC_PATH=3D\"/prefix/afl-0.60b/share/doc/afl\" -DVERSION=3D\"0.60b\"
> >> -Wno-format test-instr.c -o test-instr
> >>   /tmp/.afl-19244-1416499444.s: Assembler messages:
> >>   /tmp/.afl-19244-1416499444.s:222: Error: unknown pseudo-op: `.cfi_se=
ctions'
> >>   clang: error: assembler command failed with exit code 1 (use -v to
> >> see invocation)
> >
> > I updated the port to (hopefully) use as from ports on i386:
> > http://www.fabiankeil.de/sourcecode/freebsd/afl-61b.shar
> >
> > Does this make a difference?
>=20
> Maybe, if you want to force devel/binutils on 9.x users. It'd be nice to
> debug why clang misbehaves. Anyway, this version has wrong checksum.
>=20
>   =3D> afl-0.61b.tgz doesn't seem to exist in /portdistfiles/.
>   =3D> Attempting to fetch http://lcamtuf.coredump.cx/afl/releases/afl-0.=
61b.tgz
>   fetch: http://lcamtuf.coredump.cx/afl/releases/afl-0.61b.tgz: size mism=
atch: expected 678088, actual 678234

The tarball got re-rolled ...

> After fixing I've tested on 11.0C i386, 10.1R i386, 10.0R amd64, 9.3R i38=
6,
> 9.1R i386, 8.4R amd64 + tainted host on 11.0C amd64. For one, 8.x exhibit=
=20
> another old GNU as(1) issue:
>=20
>   [*] Testing the CC wrapper and instrumentation output...
>   AFL_QUIET=3D1 AFL_INST_RATIO=3D100 AFL_PATH=3D. ./afl-gcc -O2 -pipe -fs=
tack-protector -fno-strict-aliasing -Wall -D_FORTIFY_SOURCE=3D2 -Wno-pointe=
r-sign -DAFL_PATH=3D\"/prefix/afl-0.61b/lib/afl\" -DDOC_PATH=3D\"/prefix/af=
l-0.61b/share/doc/afl\" -DVERSION=3D\"0.61b\" -Wno-format test-instr.c -o t=
est-instr
>   /tmp/.afl-16870-1416574405.s: Assembler messages:
>   /tmp/.afl-16870-1416574405.s:572: Error: suffix or operands invalid for=
 `lahf'
>   /tmp/.afl-16870-1416574405.s:593: Error: suffix or operands invalid for=
 `sahf'
>   Makefile:65: recipe for target 'test_build' failed
>=20
> > +.if ${ARCH} =3D=3D "i386"
> > +BUILD_DEPENDS +=3D	${LOCALBASE}/bin/as:${PORTSDIR}/devel/binutils
> > +.endif
> [...]
> > +.if ${ARCH} =3D=3D "i386"
> > +	${REINPLACE_CMD} -e 's@\(  as_params\[0\] =3D "\)@\1${LOCALBASE}/bin/=
@' \
> > +		${WRKSRC}/afl-as.c
> > +.endif
>=20
> If ${LOCALBASE}/bin/as maybe called after install then you have to
> adjust RUN_DEPENDS. Keep in mind package-only users.

Indeed.

> >
> > If not, I'll probably just submit the port marked as broken
> > for i386 and try to get this working later on.

> A passing by committer may also complain about PORTVERSION vs. DISTVERSIO=
N,
> lack of LICENSE and DATADIR in pkg-plist.

I'm aware of the last two potential complaints and am prepared to
deal with them, but I'm not sure I understand the first one.

Are you suggesting that using DISTVERSION instead of PORTVERSION would
be more appropriate? At least from the comments in bsd.port.mk that's
not obvious to me.

> diff --git security/afl/Makefile security/afl/Makefile
> index e197507..db31853 100644
> --- security/afl/Makefile
> +++ security/afl/Makefile

Thanks a lot for the patch, I updated the shar file and will submit it
tomorrow provided the tarball doesn't get re-rolled again.

Fabian

--Sig_/L_6/tikl2bRELzBDQ_6m8yG
Content-Type: application/pgp-signature
Content-Description: OpenPGP digital signature

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2

iEYEARECAAYFAlRvb9QACgkQBYqIVf93VJ0+ewCgxrgUx92GXlRSa7sYTQGAUNZ3
OuYAniObuDtUWIjSJUjn9Lf03kj3P0qB
=ROI2
-----END PGP SIGNATURE-----

--Sig_/L_6/tikl2bRELzBDQ_6m8yG--



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?6f44289d.63c6d622>