withSystemRandom uses time as seed source #8

Closed
Shimuuar opened this Issue Dec 16, 2011 · 2 comments

2 participants

@Shimuuar
Collaborator

Note: on Windows, this code does not yet use the native Cryptographic API as a source of random numbers (it uses the system clock instead). As a result, the sequences it generates may not be highly independent.

P.S. I'm not affected by this bug.

@mcandre

For security's sake, it would be nice for MWC to actually use the Windows crypto API.

For CLI cleanliness, it would be nice just to get rid of those warnings.

@thoughtpolice thoughtpolice added a commit to thoughtpolice/mwc-random that referenced this issue Feb 19, 2016
@thoughtpolice thoughtpolice win32: Use RtlGenRandom for seeding
Fixes #8.

Signed-off-by: Austin Seipp <aseipp@pobox.com>
e34f2ca
@thoughtpolice thoughtpolice added a commit to thoughtpolice/mwc-random that referenced this issue Feb 19, 2016
@thoughtpolice thoughtpolice win32: Use RtlGenRandom for seeding
Fixes #8.

Signed-off-by: Austin Seipp <aseipp@pobox.com>
82bfcea
@thoughtpolice thoughtpolice added a commit to thoughtpolice/mwc-random that referenced this issue Feb 19, 2016
@thoughtpolice thoughtpolice win32: Use RtlGenRandom for seeding
Fixes #8.

Signed-off-by: Austin Seipp <aseipp@pobox.com>
c1d01f6
@thoughtpolice thoughtpolice added a commit to thoughtpolice/mwc-random that referenced this issue Feb 19, 2016
@thoughtpolice thoughtpolice win32: Use RtlGenRandom for seeding
Fixes #8.

Signed-off-by: Austin Seipp <aseipp@pobox.com>
ff31c3d
@Shimuuar Shimuuar closed this in #52 Feb 20, 2016
@Shimuuar
Collaborator

Fixed by ff31c3d

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment