Closed Bug 882829 Opened 11 years ago Closed 11 years ago

RNG_SystemRNG should not fall back on rng_systemFromNoise on Windows

Categories

(NSS :: Libraries, defect, P2)

All
Windows XP
defect

Tracking

(Not tracked)

RESOLVED FIXED
3.15.1

People

(Reporter: wtc, Assigned: wtc)

Details

Attachments

(1 file)

All the major operating systems have a system entropy source:
/dev/urandom on POSIX and RtlGenRandom on Windows.

By default, RNG_SystemRNG should fail (returning 0) rather than
falling back on rng_systemFromNoise if it cannot use the system
entropy source.

The attached Windows patch makes this change to the RNG_SystemRNG
function in lib/freebl/win_rand.c. It also removes the obsolete
code for Windows versions older than Windows XP.
Attachment #762187 - Flags: superreview?(rrelyea)
Attachment #762187 - Flags: review?(bsmith)
Comment on attachment 762187 [details] [diff] [review]
Windows patch (lib/freebl/win_rand.c)

Ryan reviewed this patch at https://codereview.chromium.org/17058006/

Patch checked in: https://hg.mozilla.org/projects/nss/rev/17df3000588a
Attachment #762187 - Flags: superreview?(rrelyea)
Attachment #762187 - Flags: review?(bsmith)
Attachment #762187 - Flags: checked-in+
Edited the bug's summary to reflect what was done in NSS 3.15.1 (Windows only).

Opened bug 889116 to finish this work.
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
OS: All → Windows XP
Resolution: --- → FIXED
Summary: RNG_SystemRNG should not fall back on rng_systemFromNoise on platforms with a system entropy source → RNG_SystemRNG should not fall back on rng_systemFromNoise on Windows
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: