Fix return code in tls1_mac
authorMatt Caswell <matt@openssl.org>
Fri, 19 May 2017 09:28:43 +0000 (10:28 +0100)
committerMatt Caswell <matt@openssl.org>
Fri, 19 May 2017 16:02:11 +0000 (17:02 +0100)
commitaabe3a359197d99867abaab8a4beb42003ee7d79
tree0c1d9f534902225c3797036e1389b4f9bdc560a8
parentffbaf06ade6dab6a0805a24087cf2e84c5db8d43
Fix return code in tls1_mac

The return code from tls1_mac is supposed to be a boolean 0 for fail, 1 for
success. In one place we returned -1 on error. This would cause code calling
the mac function to erroneously see this as a success (because a non-zero
value is being treated as success in all call sites).

Fortunately, AFAICT, the place that returns -1 can only happen on an
internal error so is not under attacker control. Additionally this code only
appears in master. In 1.1.0 the return codes are treated differently.
Therefore there are no security implications.

Reviewed-by: Rich Salz <rsalz@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/3495)
ssl/record/ssl3_record.c