diff options
author | 2017-08-11 20:56:15 +0000 | |
---|---|---|
committer | 2017-08-11 20:56:15 +0000 | |
commit | 7704014b24bf4a2c156cffa87421c8402490afb4 (patch) | |
tree | 8ce395a63f7ccc099623b7fcbc50b44f33beae62 /lib/libssl/ssl_lib.c | |
parent | Merge DDBCTF into DDB. (diff) | |
download | wireguard-openbsd-7704014b24bf4a2c156cffa87421c8402490afb4.tar.xz wireguard-openbsd-7704014b24bf4a2c156cffa87421c8402490afb4.zip |
During MPPE key reduction on the 40 bits case the first 3 octets need to be
changed with known constants (RFC3079). Current code uses a switch case without
breaks which implicitly makes the code correct, but to improve readibility the
first octect should have the constant assigned also in the first case, without
relying on a fallthrough to the second, and the break statement should be
called on boths cases.
This was a false positive found in Coverity CID 1453390, but changed due to
to readibility as explained above.
After discussion with millert@ and guenther@
Diffstat (limited to 'lib/libssl/ssl_lib.c')
0 files changed, 0 insertions, 0 deletions