diff mbox series

[v2] toolchain: musl: disable crypt size hack by default

Message ID 20211214121720.20950-1-ynezz@true.cz
State Accepted
Delegated to: Petr Štetiar
Headers show
Series [v2] toolchain: musl: disable crypt size hack by default | expand

Commit Message

Petr Štetiar Dec. 14, 2021, 12:17 p.m. UTC
Enable this option and thus re-include crypt() support for the SHA256,
SHA512 and Blowfish ciphers on all devices. According to commit
9365745f8e7b ("musl: add a hack to remove unused crypt() algorithms,
saves ~14k after lzma") it should add about ~14k to the resulting image,
which seems to be a reasonable size increase for consistent crypt()
support.

Decided to not remove this hack completely as it might be still useful
for people trying to fit custom images onto smaller devices and the
patch is rather simple so we can afford to keep it for now.

References: https://github.com/openwrt/openwrt/pull/1331
Signed-off-by: Petr Štetiar <ynezz@true.cz>
---
 Changes since v1:

  * enable full crypt() support on all devices (Jo)

 toolchain/musl/Config.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff mbox series

Patch

diff --git a/toolchain/musl/Config.in b/toolchain/musl/Config.in
index 7e83b6fa535d..a8feeffbc860 100644
--- a/toolchain/musl/Config.in
+++ b/toolchain/musl/Config.in
@@ -3,7 +3,7 @@ 
 config MUSL_DISABLE_CRYPT_SIZE_HACK
 	bool "Include crypt() support for SHA256, SHA512 and Blowfish ciphers"
 	depends on TOOLCHAINOPTS && USE_MUSL && !EXTERNAL_TOOLCHAIN
-	default n
+	default y
 	help
 	  Enable this option to re-include crypt() support for the SHA256, SHA512 and
 	  Blowfish ciphers. Without this option, attempting to hash a string with a salt