Re: CVS commit: src/usr.bin/crunch/crunchgen

2019-12-23 Thread Joerg Sonnenberger
On Mon, Dec 23, 2019 at 06:54:47PM -, Christos Zoulas wrote: > Here is a patch that: > > 1. removes all the special handling of variables (-d -p -P -s -S) that >were dealing with DBG (-d) LDSTATIC/NOPIE (-p), and the rest with >disabling/enabling sanitizers. > 2. uses emalloc/estrdup f

Re: CVS commit: src/usr.bin/crunch/crunchgen

2019-12-19 Thread Joerg Sonnenberger
On Thu, Dec 19, 2019 at 12:27:23PM -0500, Christos Zoulas wrote: > On Dec 19, 4:19pm, jo...@bec.de (Joerg Sonnenberger) wrote: > -- Subject: Re: CVS commit: src/usr.bin/crunch/crunchgen > > | > I think that there are two uses: install media and rescue. I can reinstate > | &

Re: CVS commit: src/usr.bin/crunch/crunchgen

2019-12-19 Thread Christos Zoulas
On Dec 19, 4:19pm, jo...@bec.de (Joerg Sonnenberger) wrote: -- Subject: Re: CVS commit: src/usr.bin/crunch/crunchgen | > I think that there are two uses: install media and rescue. I can reinstate | > it for rescue. Is that ok? Note that by default both ssp and fortify are | > off for mo

Re: CVS commit: src/usr.bin/crunch/crunchgen

2019-12-19 Thread Joerg Sonnenberger
On Thu, Dec 19, 2019 at 01:46:43PM -, Christos Zoulas wrote: > In article <20191218222723.ga17...@bec.de>, > Joerg Sonnenberger wrote: > >On Wed, Dec 18, 2019 at 03:51:21PM -, Christos Zoulas wrote: > >> In article <20191218152113.ga7...@bec.de>, > >> Joerg Sonnenberger wrote: > >> >On

Re: CVS commit: src/usr.bin/crunch/crunchgen

2019-12-19 Thread Christos Zoulas
In article <20191218222723.ga17...@bec.de>, Joerg Sonnenberger wrote: >On Wed, Dec 18, 2019 at 03:51:21PM -, Christos Zoulas wrote: >> In article <20191218152113.ga7...@bec.de>, >> Joerg Sonnenberger wrote: >> >On Tue, Dec 17, 2019 at 09:16:04PM -0500, Christos Zoulas wrote: >> >> Module Na

Re: CVS commit: src/usr.bin/crunch/crunchgen

2019-12-18 Thread Joerg Sonnenberger
On Wed, Dec 18, 2019 at 03:51:21PM -, Christos Zoulas wrote: > In article <20191218152113.ga7...@bec.de>, > Joerg Sonnenberger wrote: > >On Tue, Dec 17, 2019 at 09:16:04PM -0500, Christos Zoulas wrote: > >> Module Name: src > >> Committed By: christos > >> Date: Wed De

Re: CVS commit: src/usr.bin/crunch/crunchgen

2019-12-18 Thread Christos Zoulas
In article <20191218152113.ga7...@bec.de>, Joerg Sonnenberger wrote: >On Tue, Dec 17, 2019 at 09:16:04PM -0500, Christos Zoulas wrote: >> Module Name: src >> Committed By:christos >> Date:Wed Dec 18 02:16:04 UTC 2019 >> >> Modified Files: >> src/usr.bin/crunch/crunch

Re: CVS commit: src/usr.bin/crunch/crunchgen

2019-12-18 Thread Joerg Sonnenberger
On Tue, Dec 17, 2019 at 09:16:04PM -0500, Christos Zoulas wrote: > Module Name: src > Committed By: christos > Date: Wed Dec 18 02:16:04 UTC 2019 > > Modified Files: > src/usr.bin/crunch/crunchgen: crunchgen.1 crunchgen.c > > Log Message: > Also disable ssp and fortify by default.

Re: CVS commit: src/usr.bin/crunch/crunchgen

2019-02-17 Thread Joerg Sonnenberger
On Wed, Feb 13, 2019 at 07:56:26AM +0100, Martin Husemann wrote: > On Tue, Feb 12, 2019 at 09:20:23PM -, Christos Zoulas wrote: > > Well, regardless of what the right permissions of .eh_frame are, > > we could just nuke the code and default to the "new" behavior... > > We can then put back the

re: CVS commit: src/usr.bin/crunch/crunchgen

2019-02-13 Thread matthew green
> >> The real bug is the reverted varasm.c change. GCC creates the .eh_frame > >> section with the wrong permissions. > > > >yes - putting your varasm.c back fixes the crtbegin.o > >build, but it breaks the libstdc++ one: > > > >In file included from > >/usr/src4/external/gpl3/gcc/dist/libstdc++-v3

Re: CVS commit: src/usr.bin/crunch/crunchgen

2019-02-13 Thread Christos Zoulas
In article <20190213065626.ga22...@mail.duskware.de>, Martin Husemann wrote: >On Tue, Feb 12, 2019 at 09:20:23PM -, Christos Zoulas wrote: >> Well, regardless of what the right permissions of .eh_frame are, >> we could just nuke the code and default to the "new" behavior... >> We can then put

Re: CVS commit: src/usr.bin/crunch/crunchgen

2019-02-12 Thread Martin Husemann
On Tue, Feb 12, 2019 at 09:20:23PM -, Christos Zoulas wrote: > Well, regardless of what the right permissions of .eh_frame are, > we could just nuke the code and default to the "new" behavior... > We can then put back the varasm.c change... > This of course needs to be evaluated carefully. Why

Re: CVS commit: src/usr.bin/crunch/crunchgen

2019-02-12 Thread Christos Zoulas
In article <16690.1550005...@splode.eterna.com.au>, matthew green wrote: >Joerg Sonnenberger writes: >> On Tue, Feb 12, 2019 at 10:16:58AM +, matthew green wrote: >> > Module Name: src >> > Committed By: mrg >> > Date: Tue Feb 12 10:16:58 UTC 2019 >> > >> > Modified F

re: CVS commit: src/usr.bin/crunch/crunchgen

2019-02-12 Thread matthew green
Joerg Sonnenberger writes: > On Tue, Feb 12, 2019 at 10:16:58AM +, matthew green wrote: > > Module Name:src > > Committed By: mrg > > Date: Tue Feb 12 10:16:58 UTC 2019 > > > > Modified Files: > > src/usr.bin/crunch/crunchgen: crunchgen.c > > > > Log Message: >

Re: CVS commit: src/usr.bin/crunch/crunchgen

2019-02-12 Thread Joerg Sonnenberger
On Tue, Feb 12, 2019 at 10:16:58AM +, matthew green wrote: > Module Name: src > Committed By: mrg > Date: Tue Feb 12 10:16:58 UTC 2019 > > Modified Files: > src/usr.bin/crunch/crunchgen: crunchgen.c > > Log Message: > hack alert time: > > on sparc and sparc64, don't remove .eh