lkml.org 
[lkml]   [2017]   [Oct]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [PATCH 2/2] crypto/drbg: account for no longer returning -EALREADY
Date
Am Donnerstag, 19. Oktober 2017, 22:45:06 CEST schrieb Jason A. Donenfeld:

Hi Jason,

> We now structure things in a way that assumes the seeding function is
> always eventually called.
>
> Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
> ---
> crypto/drbg.c | 20 +++++---------------
> 1 file changed, 5 insertions(+), 15 deletions(-)
>
> diff --git a/crypto/drbg.c b/crypto/drbg.c
> index 70018397e59a..501e013cb96c 100644
> --- a/crypto/drbg.c
> +++ b/crypto/drbg.c
> @@ -1411,18 +1411,8 @@ static int drbg_prepare_hrng(struct drbg_state *drbg)
>
> err = add_random_ready_callback(&drbg->random_ready);
>
> - switch (err) {
> - case 0:
> - break;
> -
> - case -EALREADY:
> - err = 0;
> - /* fall through */
> -
> - default:
> - drbg->random_ready.func = NULL;
> + if (err)
> return err;

Don't you change the logic flow here? In case the add_random_ready_callback
returns 0 because the ready function is already called due to crng_ready(),
the new code above in the patch set continues. But with the current code, it
will return at this point and do not perform the allocation below.



Ciao
Stephan

\
 
 \ /
  Last update: 2017-10-22 17:51    [W:0.051 / U:5.620 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site