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

Petr Štetiar ynezz at true.cz
Tue Dec 14 04:17:20 PST 2021


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 at 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 --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



More information about the openwrt-devel mailing list