svn commit: r345875 - in head: contrib/elftoolchain/strings usr.bin/strings
Adrian Chadd
adrian.chadd at gmail.com
Wed Apr 17 16:53:27 UTC 2019
not yet? I'm still debugging other issues; this just pissed off gcc. :)
-a
On Sun, 14 Apr 2019 at 19:42, Mariusz Zaborski <oshogbo at freebsd.org> wrote:
> On Sun, Apr 14, 2019 at 08:19:22AM -0700, Adrian Chadd wrote:
> > This exposed a fun bug - gcc mips 6.4.0 complains about an unused arg
> (fa)
> > in fileargs_fopen() in the strings change when you compile without casper
> > support. I do that on mips.
> >
> > I have a local change that converts the #define to an inline function so
> > the unused arg can be (void)'ed away. Mind if I commit it?
> Sounds good to me.
> Although you don't have an issue with others services?
>
> --
> Mariusz Zaborski
> oshogbo//vx | http://oshogbo.vexillium.org
> FreeBSD committer | https://freebsd.org
> Software developer | http://wheelsystems.com
> If it's not broken, let's fix it till it is!!1
>
> >
> > On Thu, 4 Apr 2019 at 09:32, Mariusz Zaborski <oshogbo at freebsd.org>
> wrote:
> >
> > > Author: oshogbo
> > > Date: Thu Apr 4 16:32:27 2019
> > > New Revision: 345875
> > > URL: https://svnweb.freebsd.org/changeset/base/345875
> > >
> > > Log:
> > > strings: capsicumize it
> > >
> > > Reviewed by: cem
> > > Discussed with: emaste
> > > Differential Revision: https://reviews.freebsd.org/D18038
> > >
> > > Modified:
> > > head/contrib/elftoolchain/strings/strings.c
> > > head/usr.bin/strings/Makefile
> > >
> > > Modified: head/contrib/elftoolchain/strings/strings.c
> > >
> > >
> ==============================================================================
> > > --- head/contrib/elftoolchain/strings/strings.c Thu Apr 4 12:02:48
> 2019
> > > (r345874)
> > > +++ head/contrib/elftoolchain/strings/strings.c Thu Apr 4 16:32:27
> 2019
> > > (r345875)
> > > @@ -25,8 +25,10 @@
> > > */
> > >
> > > #include <sys/types.h>
> > > +#include <sys/capsicum.h>
> > > #include <sys/stat.h>
> > >
> > > +#include <capsicum_helpers.h>
> > > #include <ctype.h>
> > > #include <err.h>
> > > #include <errno.h>
> > > @@ -44,6 +46,9 @@
> > > #include <libelftc.h>
> > > #include <gelf.h>
> > >
> > > +#include <libcasper.h>
> > > +#include <casper/cap_fileargs.h>
> > > +
> > > #include "_elftc.h"
> > >
> > > ELFTC_VCSID("$Id: strings.c 3648 2018-11-22 23:26:43Z emaste $");
> > > @@ -85,7 +90,7 @@ static struct option strings_longopts[] = {
> > > };
> > >
> > > int getcharacter(FILE *, long *);
> > > -int handle_file(const char *);
> > > +int handle_file(fileargs_t *fa, const char *);
> > > int handle_elf(const char *, FILE *);
> > > int handle_binary(const char *, FILE *, size_t);
> > > int find_strings(const char *, FILE *, off_t, off_t);
> > > @@ -99,6 +104,8 @@ void usage(void);
> > > int
> > > main(int argc, char **argv)
> > > {
> > > + fileargs_t *fa;
> > > + cap_rights_t rights;
> > > int ch, rc;
> > >
> > > rc = 0;
> > > @@ -187,27 +194,41 @@ main(int argc, char **argv)
> > > argc -= optind;
> > > argv += optind;
> > >
> > > + cap_rights_init(&rights, CAP_READ, CAP_SEEK, CAP_FSTAT,
> CAP_FCNTL);
> > > + fa = fileargs_init(argc, argv, O_RDONLY, 0, &rights);
> > > + if (fa == NULL)
> > > + err(1, "Unable to initialize casper fileargs");
> > > +
> > > + caph_cache_catpages();
> > > + if (caph_limit_stdio() < 0 && caph_enter_casper() < 0) {
> > > + fileargs_free(fa);
> > > + err(1, "Unable to enter capability mode");
> > > + }
> > > +
> > > if (min_len == 0)
> > > min_len = 4;
> > > if (*argv == NULL)
> > > rc = find_strings("{standard input}", stdin, 0, 0);
> > > else while (*argv != NULL) {
> > > - if (handle_file(*argv) != 0)
> > > + if (handle_file(fa, *argv) != 0)
> > > rc = 1;
> > > argv++;
> > > }
> > > +
> > > + fileargs_free(fa);
> > > +
> > > return (rc);
> > > }
> > >
> > > int
> > > -handle_file(const char *name)
> > > +handle_file(fileargs_t *fa, const char *name)
> > > {
> > > FILE *pfile;
> > > int rt;
> > >
> > > if (name == NULL)
> > > return (1);
> > > - pfile = fopen(name, "rb");
> > > + pfile = fileargs_fopen(fa, name, "rb");
> > > if (pfile == NULL) {
> > > warnx("'%s': %s", name, strerror(errno));
> > > return (1);
> > >
> > > Modified: head/usr.bin/strings/Makefile
> > >
> > >
> ==============================================================================
> > > --- head/usr.bin/strings/Makefile Thu Apr 4 12:02:48 2019
> > > (r345874)
> > > +++ head/usr.bin/strings/Makefile Thu Apr 4 16:32:27 2019
> > > (r345875)
> > > @@ -10,6 +10,12 @@ PROG= strings
> > >
> > > LIBADD= elftc elf
> > >
> > > +.if ${MK_CASPER} != "no" && !defined(BOOTSTRAPPING)
> > > +LIBADD+= casper
> > > +LIBADD+= cap_fileargs
> > > +CFLAGS+= -DWITH_CASPER
> > > +.endif
> > > +
> > > CFLAGS+=-I${ELFTCDIR}/libelftc -I${ELFTCDIR}/common
> > >
> > > .include <bsd.prog.mk>
> > >
> > >
>
More information about the svn-src-all
mailing list