* [Buildroot] [PATCH] utils/genrandconfig: drop hardening Config enables
@ 2021-05-21 13:17 Matthew Weber
2021-05-24 13:16 ` Yann E. MORIN
2021-05-24 13:31 ` Yann E. MORIN
0 siblings, 2 replies; 4+ messages in thread
From: Matthew Weber @ 2021-05-21 13:17 UTC (permalink / raw)
To: buildroot
Since 810ba387bec3c5b, some form of these options are enable
by default. Specifically:
- Kept FORTIFY level 2 option as the default is now level 1.
- Removed all SSP options as the default now uses the best
option based on toolchain support.
- Similar to SSP, for RELRO, the default now uses the best
option based on toolchain support.
- Completely drop PIC PIE as it defaults =y
Signed-off-by: Matthew Weber <matthew.weber@collins.com>
---
utils/genrandconfig | 14 --------------
1 file changed, 14 deletions(-)
diff --git a/utils/genrandconfig b/utils/genrandconfig
index 93dc6d898b..e1655655fa 100755
--- a/utils/genrandconfig
+++ b/utils/genrandconfig
@@ -371,22 +371,8 @@ def gen_config(args):
configlines.append("BR2_OPTIMIZE_2=y\n")
if randint(0, 4) == 0:
configlines.append("BR2_SYSTEM_ENABLE_NLS=y\n")
- if randint(0, 4) == 0:
- configlines.append("BR2_PIC_PIE=y\n")
- if randint(0, 4) == 0:
- configlines.append("BR2_RELRO_FULL=y\n")
- elif randint(0, 4) == 0:
- configlines.append("BR2_RELRO_PARTIAL=y\n")
- if randint(0, 4) == 0:
- configlines.append("BR2_SSP_ALL=y\n")
- elif randint(0, 4) == 0:
- configlines.append("BR2_SSP_REGULAR=y\n")
- elif randint(0, 4) == 0:
- configlines.append("BR2_SSP_STRONG=y\n")
if randint(0, 4) == 0:
configlines.append("BR2_FORTIFY_SOURCE_2=y\n")
- elif randint(0, 4) == 0:
- configlines.append("BR2_FORTIFY_SOURCE_1=y\n")
# Randomly enable BR2_REPRODUCIBLE 10% of times
# also enable tar filesystem images for testing
--
2.17.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [Buildroot] [PATCH] utils/genrandconfig: drop hardening Config enables
2021-05-21 13:17 [Buildroot] [PATCH] utils/genrandconfig: drop hardening Config enables Matthew Weber
@ 2021-05-24 13:16 ` Yann E. MORIN
2021-05-24 13:32 ` Yann E. MORIN
2021-05-24 13:31 ` Yann E. MORIN
1 sibling, 1 reply; 4+ messages in thread
From: Yann E. MORIN @ 2021-05-24 13:16 UTC (permalink / raw)
To: buildroot
Matthew, All,
On 2021-05-21 08:17 -0500, Matthew Weber via buildroot spake thusly:
> Since 810ba387bec3c5b, some form of these options are enable
> by default. Specifically:
>
> - Kept FORTIFY level 2 option as the default is now level 1.
> - Removed all SSP options as the default now uses the best
> option based on toolchain support.
> - Similar to SSP, for RELRO, the default now uses the best
> option based on toolchain support.
> - Completely drop PIC PIE as it defaults =y
>
> Signed-off-by: Matthew Weber <matthew.weber@collins.com>
> ---
> utils/genrandconfig | 14 --------------
> 1 file changed, 14 deletions(-)
>
> diff --git a/utils/genrandconfig b/utils/genrandconfig
> index 93dc6d898b..e1655655fa 100755
> --- a/utils/genrandconfig
> +++ b/utils/genrandconfig
> @@ -371,22 +371,8 @@ def gen_config(args):
> configlines.append("BR2_OPTIMIZE_2=y\n")
> if randint(0, 4) == 0:
> configlines.append("BR2_SYSTEM_ENABLE_NLS=y\n")
> - if randint(0, 4) == 0:
> - configlines.append("BR2_PIC_PIE=y\n")
> - if randint(0, 4) == 0:
> - configlines.append("BR2_RELRO_FULL=y\n")
> - elif randint(0, 4) == 0:
> - configlines.append("BR2_RELRO_PARTIAL=y\n")
> - if randint(0, 4) == 0:
> - configlines.append("BR2_SSP_ALL=y\n")
> - elif randint(0, 4) == 0:
> - configlines.append("BR2_SSP_REGULAR=y\n")
> - elif randint(0, 4) == 0:
> - configlines.append("BR2_SSP_STRONG=y\n")
> if randint(0, 4) == 0:
> configlines.append("BR2_FORTIFY_SOURCE_2=y\n")
> - elif randint(0, 4) == 0:
> - configlines.append("BR2_FORTIFY_SOURCE_1=y\n")
I was going to apply this, but then I was wondering: shouldn't we
randomly *disable* them, now?
Indeed, as much enabling those options randomly helped uncover build
issues, maybe disabling them would also help?
But I don't feel very strong either, so you should have no issue
convincing me to apply as-is. ;-)
Regards,
Yann E. MORIN.
> # Randomly enable BR2_REPRODUCIBLE 10% of times
> # also enable tar filesystem images for testing
> --
> 2.17.1
>
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 561 099 427 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
^ permalink raw reply [flat|nested] 4+ messages in thread
* [Buildroot] [PATCH] utils/genrandconfig: drop hardening Config enables
2021-05-21 13:17 [Buildroot] [PATCH] utils/genrandconfig: drop hardening Config enables Matthew Weber
2021-05-24 13:16 ` Yann E. MORIN
@ 2021-05-24 13:31 ` Yann E. MORIN
1 sibling, 0 replies; 4+ messages in thread
From: Yann E. MORIN @ 2021-05-24 13:31 UTC (permalink / raw)
To: buildroot
Matthew, All,
On 2021-05-21 08:17 -0500, Matthew Weber via buildroot spake thusly:
> Since 810ba387bec3c5b, some form of these options are enable
> by default. Specifically:
>
> - Kept FORTIFY level 2 option as the default is now level 1.
> - Removed all SSP options as the default now uses the best
> option based on toolchain support.
> - Similar to SSP, for RELRO, the default now uses the best
> option based on toolchain support.
> - Completely drop PIC PIE as it defaults =y
>
> Signed-off-by: Matthew Weber <matthew.weber@collins.com>
Applied to master, thanks.
Regards,
Yann E. MORIN.
> ---
> utils/genrandconfig | 14 --------------
> 1 file changed, 14 deletions(-)
>
> diff --git a/utils/genrandconfig b/utils/genrandconfig
> index 93dc6d898b..e1655655fa 100755
> --- a/utils/genrandconfig
> +++ b/utils/genrandconfig
> @@ -371,22 +371,8 @@ def gen_config(args):
> configlines.append("BR2_OPTIMIZE_2=y\n")
> if randint(0, 4) == 0:
> configlines.append("BR2_SYSTEM_ENABLE_NLS=y\n")
> - if randint(0, 4) == 0:
> - configlines.append("BR2_PIC_PIE=y\n")
> - if randint(0, 4) == 0:
> - configlines.append("BR2_RELRO_FULL=y\n")
> - elif randint(0, 4) == 0:
> - configlines.append("BR2_RELRO_PARTIAL=y\n")
> - if randint(0, 4) == 0:
> - configlines.append("BR2_SSP_ALL=y\n")
> - elif randint(0, 4) == 0:
> - configlines.append("BR2_SSP_REGULAR=y\n")
> - elif randint(0, 4) == 0:
> - configlines.append("BR2_SSP_STRONG=y\n")
> if randint(0, 4) == 0:
> configlines.append("BR2_FORTIFY_SOURCE_2=y\n")
> - elif randint(0, 4) == 0:
> - configlines.append("BR2_FORTIFY_SOURCE_1=y\n")
>
> # Randomly enable BR2_REPRODUCIBLE 10% of times
> # also enable tar filesystem images for testing
> --
> 2.17.1
>
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 561 099 427 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
^ permalink raw reply [flat|nested] 4+ messages in thread
* [Buildroot] [PATCH] utils/genrandconfig: drop hardening Config enables
2021-05-24 13:16 ` Yann E. MORIN
@ 2021-05-24 13:32 ` Yann E. MORIN
0 siblings, 0 replies; 4+ messages in thread
From: Yann E. MORIN @ 2021-05-24 13:32 UTC (permalink / raw)
To: buildroot
Matthew, All,
On 2021-05-24 15:16 +0200, Yann E. MORIN spake thusly:
> On 2021-05-21 08:17 -0500, Matthew Weber via buildroot spake thusly:
> > Since 810ba387bec3c5b, some form of these options are enable
> > by default. Specifically:
> >
> > - Kept FORTIFY level 2 option as the default is now level 1.
> > - Removed all SSP options as the default now uses the best
> > option based on toolchain support.
> > - Similar to SSP, for RELRO, the default now uses the best
> > option based on toolchain support.
> > - Completely drop PIC PIE as it defaults =y
> >
> > Signed-off-by: Matthew Weber <matthew.weber@collins.com>
> > ---
[--SNIP--]
> I was going to apply this, but then I was wondering: shouldn't we
> randomly *disable* them, now?
>
> Indeed, as much enabling those options randomly helped uncover build
> issues, maybe disabling them would also help?
>
> But I don't feel very strong either, so you should have no issue
> convincing me to apply as-is. ;-)
As discussed on IRC: if there is an issue with any of those disabled, we
can handle them as they are reported by users.
Regards,
Yann E. MORIN.
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 561 099 427 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2021-05-24 13:32 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-05-21 13:17 [Buildroot] [PATCH] utils/genrandconfig: drop hardening Config enables Matthew Weber
2021-05-24 13:16 ` Yann E. MORIN
2021-05-24 13:32 ` Yann E. MORIN
2021-05-24 13:31 ` Yann E. MORIN
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.