simon-git: putty (master): Simon Tatham
Commits to Tartarus hosted VCS
tartarus-commits at lists.tartarus.org
Sun May 5 20:30:55 BST 2019
TL;DR:
4fb20b15 Move random_save_seed() into sshrand.c.
03aeabfb Use a proper PRNG for GTK askpass.
Repository: https://git.tartarus.org/simon/putty.git
On the web: https://git.tartarus.org/?p=simon/putty.git
Branch updated: master
Committer: Simon Tatham <anakin at pobox.com>
Date: 2019-05-05 20:30:55
commit 4fb20b15f31600143a111659b4b0ce1ee5ba4ea4
web diff https://git.tartarus.org/?p=simon/putty.git;a=commitdiff;h=4fb20b15f31600143a111659b4b0ce1ee5ba4ea4;hp=5f35f5b4aca8eb879e374cfd9370ac7b0489cf94
Author: Simon Tatham <anakin at pobox.com>
Date: Sun May 5 19:59:35 2019 +0100
Move random_save_seed() into sshrand.c.
It's identical in uxnoise and winnoise, being written entirely in
terms of existing cross-platform functions. Might as well centralise
it into sshrand.c.
sshrand.c | 13 +++++++++++++
unix/uxnoise.c | 12 ------------
windows/winnoise.c | 12 ------------
3 files changed, 13 insertions(+), 24 deletions(-)
commit 03aeabfbeaa88e802c966e90a425c0af2ccfaca1
web diff https://git.tartarus.org/?p=simon/putty.git;a=commitdiff;h=03aeabfbeaa88e802c966e90a425c0af2ccfaca1;hp=4fb20b15f31600143a111659b4b0ce1ee5ba4ea4
Author: Simon Tatham <anakin at pobox.com>
Date: Sun May 5 20:22:36 2019 +0100
Use a proper PRNG for GTK askpass.
Coverity complained that it was wrong to use rand() in a security
context, and although in this case it's _very_ marginal, I can't
actually disagree that the choice of which light to light up to avoid
giving information about passphrase length is a security context.
So, no more rand(); instead we instantiate a shiny Fortuna PRNG
instance, seed it in more or less the usual way, and use that as an
overkill-level method of choosing which light to light up next.
(Acknowledging that this is a slightly unusual application and less
critical than most, I don't actually put the passphrase characters
themselves into the PRNG, and I don't use a random-seed file.)
Recipe | 2 +-
unix/gtkask.c | 40 +++++++++++++++++++++++++++++++++++-----
unix/uxpgnt.c | 1 -
3 files changed, 36 insertions(+), 7 deletions(-)
More information about the tartarus-commits
mailing list