diff options
| author | David Benjamin <davidben@chromium.org> | 2014-12-16 15:48:10 +0300 |
|---|---|---|
| committer | Adam Langley <agl@google.com> | 2014-12-16 22:15:59 +0300 |
| commit | a6d81018f8fd5647d88a49923633f29dd77c2365 (patch) | |
| tree | a2182160a477558c1919642629a684046938c42b /ssl/t1_enc.c | |
| parent | 263eac02f5c27ad91c1514c93246b84980f73c97 (diff) | |
Consistently use RAND_bytes and check for failure.
RAND_pseudo_bytes just calls RAND_bytes now and only returns 0 or 1. Switch all
callers within the library call the new one and use the simpler failure check.
This fixes a few error checks that no longer work (< 0) and some missing ones.
Change-Id: Id51c79deec80075949f73fa1fbd7b76aac5570c6
Reviewed-on: https://boringssl-review.googlesource.com/2621
Reviewed-by: Adam Langley <agl@google.com>
Diffstat (limited to 'ssl/t1_enc.c')
| -rw-r--r-- | ssl/t1_enc.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/ssl/t1_enc.c b/ssl/t1_enc.c index f5a4b9f6..28405b70 100644 --- a/ssl/t1_enc.c +++ b/ssl/t1_enc.c @@ -895,7 +895,7 @@ int tls1_enc(SSL *s, int send) fprintf(stderr, "%s:%d: rec->data != rec->input\n", __FILE__, __LINE__); - else if (RAND_bytes(rec->input, ivlen) <= 0) + else if (!RAND_bytes(rec->input, ivlen)) return -1; } } |