Remove some TODO(OpenSSL1.2) references
authorMatt Caswell <matt@openssl.org>
Mon, 1 Feb 2021 15:45:44 +0000 (15:45 +0000)
committerMatt Caswell <matt@openssl.org>
Thu, 4 Feb 2021 12:26:36 +0000 (12:26 +0000)
commit08cea586c9d0fd2fcf99ec1eacb7736a34139d8b
treeb40a73df67a269514b4c34e8fd1bb57acdef0fe5
parenta7246ea645b5d4c5ca7bde3dad4fcd6e63e11896
Remove some TODO(OpenSSL1.2) references

We had a couple of stray references to OpenSSL1.2 in libssl. We just
reword the comments to remove those references without changing any
behaviour.

The first one in t1_lib.c is a technical non-compliance in the TLSv1.3
spec where, under some circumstances, we offer DSA sigalgs even in a
ClientHello that eventually negotiates TLSv1.3. We explicitly chose to
accept this behaviour in 1.1.1 and we're not planning to change it for
3.0.

The second one in s3_lib.c is regarnding the behaviour of
SSL_set_tlsext_host_name(). Technically you shouldn't be able to call
this from a server - but we allow it and just ignore it rather than
raising an error. The TODO suggest we consider raising an error instead.
However, with 3.0 we are trying to minimise breaking changes so I suggest
not making this change now.

Fixes #13161

Reviewed-by: Matthias St. Pierre <Matthias.St.Pierre@ncp-e.com>
(Merged from https://github.com/openssl/openssl/pull/14037)
ssl/s3_lib.c
ssl/t1_lib.c