Merge lp:~mdeslaur/ubiquity/fix-random-seed into lp:ubiquity

Proposed by Marc Deslauriers on 2015-12-09
Status: Merged
Approved by: Mathieu Trudel-Lapierre on 2016-02-24
Approved revision: 6353
Merged at revision: 6382
Proposed branch: lp:~mdeslaur/ubiquity/fix-random-seed
Merge into: lp:ubiquity
Diff against target: 36 lines (+9/-2)
2 files modified
debian/changelog (+7/-0)
scripts/plugininstall.py (+2/-2)
To merge this branch: bzr merge lp:~mdeslaur/ubiquity/fix-random-seed
Reviewer Review Type Date Requested Status
Mathieu Trudel-Lapierre 2016-02-01 Approve on 2016-02-01
Dimitri John Ledkov 2015-12-09 Needs Information on 2015-12-09
Review via email: mp+280018@code.launchpad.net

Description of the Change

  scripts/plugininstall.py: Write initial random seed to systemd
  location (LP: #1523199)

To post a comment you must log in.
Dimitri John Ledkov (xnox) wrote :

As of 4.2 kernel there is no need for random seed. Thus systemd seed service will do nothing, and we shouldn't need to fix this in xenial.

Maybe this needs to be fixed in prior releases, i do not know.

http://lists.freedesktop.org/archives/systemd-devel/2015-June/033297.html

http://www.phoronix.com/scan.php?page=news_item&px=Linux-4.2-Crypto-Akcipher-PKE

http://marc.info/?l=linux-kernel&m=143496272614455&w=2

Thus why should we fix this?! I think we should like remove it completely.

review: Needs Information
Marc Deslauriers (mdeslaur) wrote :

The Jitter RNG linked above is for the Crypto API code, and isn't relevant for the random device.
The random device still needs to be seeded at boot.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/changelog'
2--- debian/changelog 2015-11-02 03:21:38 +0000
3+++ debian/changelog 2015-12-09 13:25:54 +0000
4@@ -1,3 +1,10 @@
5+ubiquity (2.21.39) UNRELEASED; urgency=medium
6+
7+ * scripts/plugininstall.py: Write initial random seed to systemd
8+ location (LP: #1523199)
9+
10+ -- Marc Deslauriers <marc.deslauriers@ubuntu.com> Wed, 09 Dec 2015 08:17:25 -0500
11+
12 ubiquity (2.21.38) xenial; urgency=medium
13
14 * d-i/sources.list: switch to xenial.
15
16=== modified file 'scripts/plugininstall.py'
17--- scripts/plugininstall.py 2015-11-02 03:19:11 +0000
18+++ scripts/plugininstall.py 2015-12-09 13:25:54 +0000
19@@ -1725,7 +1725,7 @@
20 return
21 if not stat.S_ISCHR(st.st_mode):
22 return
23- if not os.path.isdir(self.target_file("var/lib/urandom")):
24+ if not os.path.isdir(self.target_file("var/lib/systemd")):
25 return
26
27 poolbytes = 512
28@@ -1740,7 +1740,7 @@
29 old_umask = os.umask(0o077)
30 try:
31 with open("/dev/urandom", "rb") as urandom:
32- with open(self.target_file("var/lib/urandom/random-seed"),
33+ with open(self.target_file("var/lib/systemd/random-seed"),
34 "wb") as seed:
35 seed.write(urandom.read(poolbytes))
36 except IOError:

Subscribers

People subscribed via source and target branches

to status/vote changes: