simon-git: putty (main): Simon Tatham

Commits to Tartarus hosted VCS tartarus-commits at lists.tartarus.org
Sun May 28 10:15:23 BST 2023


TL;DR:
  8cf372d4 NTRU: remove a pointless failure check.

Repository:     https://git.tartarus.org/simon/putty.git
On the web:     https://git.tartarus.org/?p=simon/putty.git
Branch updated: main
Committer:      Simon Tatham <anakin at pobox.com>
Date:           2023-05-28 10:15:23

commit 8cf372d4a215584a031cfabbee34cbbf15391b1c
web diff https://git.tartarus.org/?p=simon/putty.git;a=commitdiff;h=8cf372d4a215584a031cfabbee34cbbf15391b1c;hp=6aca7f1eef4ad4dd41d58424917fe6f2234b8b13
Author: Simon Tatham <anakin at pobox.com>
Date:   Sun May 28 09:59:41 2023 +0100

    NTRU: remove a pointless failure check.
    
    In the key generation step where we invert 3f in the field
    Z_q/<x^p-x-1>, I was carefully checking for failure, on the grounds
    that even a field does have _one_ non-invertible element, namely zero.
    But I forgot that we'd generated f in such a way that it can't
    possibly be zero. So that failure check is pointless.
    
    (However, I've retained it in the form of an assertion.)

 crypto/ntru.c | 18 ++++++------------
 1 file changed, 6 insertions(+), 12 deletions(-)



More information about the tartarus-commits mailing list