[source] base-files: seed /dev/urandom

LEDE Commits lede-commits at lists.infradead.org
Thu Jun 30 21:44:48 PDT 2016


blogic pushed a commit to source.git, branch master:
https://git.lede-project.org/?p=source.git;a=commitdiff;h=3946a5529132c80793a9e5ee665a3cd6b0835310

commit 3946a5529132c80793a9e5ee665a3cd6b0835310
Author: Etienne CHAMPETIER <champetier.etienne at gmail.com>
AuthorDate: Mon Jun 27 15:53:30 2016 +0000

    base-files: seed /dev/urandom
    
    This commit:
    1) seed /dev/urandom with the saved seeds as early as possible
       (see /lib/preinit/81_urandom_seed)
    2) save a seed at /etc/urandom.seed if it doesn't exists
    3) save a new seed each boot at "system. at system[0].urandom_seed"
       (see /etc/init.d/urandom_seed)
    
    We use getrandom() so we are sure /dev/urandom pool is initialized
    
    Seed size is 512 bytes (ie /proc/sys/kernel/random/poolsize / 8)
    it's the same size as in ubuntu 14.04 and all systemd systems
    
    Seeding /dev/urandom doesn't change entropy estimation, so we still have
    "random: ubus urandom read with 4 bits of entropy available"
    messages in the logs, but we can now ignore them if
    after "urandom-seed: Seeding with ..." message
    
    Saving a new seed on each boot is disabled by default to avoid too much
    writes without user consent
    
    v2: log preinit messages to /dev/kmsg
    v3: use non generic function name for logging, as /lib/preinit/ files
        are all sourced together in /etc/preinit
    v4: after a lot of discussion on the ML, use a uci config param
    v5: config param is now the path of the seed
    
    Signed-off-by: Etienne CHAMPETIER <champetier.etienne at gmail.com>
    Acked-by: Jo-Philipp Wich <jo at mein.io>
---
 package/base-files/files/bin/config_generate       |  1 +
 package/base-files/files/etc/init.d/urandom_seed   | 29 ++++++++++++++++++++++
 .../base-files/files/lib/preinit/81_urandom_seed   | 24 ++++++++++++++++++
 3 files changed, 54 insertions(+)

diff --git a/package/base-files/files/bin/config_generate b/package/base-files/files/bin/config_generate
index 22cc157..7f34eec 100755
--- a/package/base-files/files/bin/config_generate
+++ b/package/base-files/files/bin/config_generate
@@ -230,6 +230,7 @@ generate_static_system() {
 		set system. at system[-1].timezone='UTC'
 		set system. at system[-1].ttylogin='0'
 		set system. at system[-1].log_size='64'
+		set system. at system[-1].urandom_seed='0'
 
 		delete system.ntp
 		set system.ntp='timeserver'
diff --git a/package/base-files/files/etc/init.d/urandom_seed b/package/base-files/files/etc/init.d/urandom_seed
new file mode 100755
index 0000000..cb2eb44
--- /dev/null
+++ b/package/base-files/files/etc/init.d/urandom_seed
@@ -0,0 +1,29 @@
+#!/bin/sh /etc/rc.common
+
+START=99
+
+EXTRA_COMMANDS="save"
+
+_log() {
+    logger -t urandom_seed "$1"
+}
+
+_save() {
+    touch $1.tmp || { _log "touch $1 failed"; return; }
+    chown root:root $1.tmp || { _log "chown $1 failed"; return; }
+    chmod 600 $1.tmp || { _log "chmod $1 failed"; return; }
+    getrandom 512 > $1.tmp || { _log "getrandom failed"; return; }
+    mv $1.tmp $1 || { _log "mv $1 failed"; return; }
+}
+
+save() {
+    SEED="$(uci -q get system. at system[0].urandom_seed)"
+    [ "${SEED:0:1}" == "/" ] && _save "$SEED" && _log "Seed saved ($SEED)"
+
+    SEED=/etc/urandom.seed
+    [ ! -f $SEED ] && _save "$SEED" && _log "Seed saved ($SEED)"
+}
+
+boot() {
+    save
+}
diff --git a/package/base-files/files/lib/preinit/81_urandom_seed b/package/base-files/files/lib/preinit/81_urandom_seed
new file mode 100644
index 0000000..10878f3
--- /dev/null
+++ b/package/base-files/files/lib/preinit/81_urandom_seed
@@ -0,0 +1,24 @@
+#!/bin/sh
+
+log_urandom_seed() {
+    echo "urandom-seed: $1" > /dev/kmsg
+}
+
+_do_urandom_seed() {
+    [ -f "$1" ] || { log_urandom_seed "Seed file not found ($1)"; return; }
+    [ -O "$1" -a -G "$1" -a ! -x "$1" ] || { log_urandom_seed "Wrong owner / permissions for $1"; return; }
+
+    log_urandom_seed "Seeding with $1"
+    cat "$1" > /dev/urandom
+}
+
+do_urandom_seed() {
+    [ -c /dev/urandom ] || { log_urandom_seed "Something is wrong with /dev/urandom"; return; }
+
+    _do_urandom_seed "/etc/urandom.seed"
+
+    SEED="$(uci -q get system. at system[0].urandom_seed)"
+    [ "${SEED:0:1}" == "/" -a "$SEED" != "/etc/urandom.seed" ] && _do_urandom_seed "$SEED"
+}
+
+boot_hook_add preinit_main do_urandom_seed



More information about the lede-commits mailing list