test/recipes/80-test_cmp_http.t: Don't trust $server_port in start_mock_server()
authorRichard Levitte <levitte@openssl.org>
Wed, 2 Jun 2021 19:19:18 +0000 (21:19 +0200)
committerRichard Levitte <levitte@openssl.org>
Sun, 6 Jun 2021 05:34:44 +0000 (07:34 +0200)
commit97cf9b05fa1cdb8e4e7f60016aa95ae0e976e8c3
treea5dfa4a02d9361d985c6109d8b67e7ee2314f3f9
parentd00be9f38760b5f143a7cdecf6c61ad6316f4cc8
test/recipes/80-test_cmp_http.t: Don't trust $server_port in start_mock_server()

Even if $server_port isn't touched, it's still a number coming from
configuration.  It's therefore not trustable as an indicator that the
ACCEPT line delivered a port number or an error indication.

$accept_msg is used instead to capture the port if there is one, and
be a better indicator of error.

Fixes #15557
Fixes #15571

Reviewed-by: David von Oheimb <david.von.oheimb@siemens.com>
(Merged from https://github.com/openssl/openssl/pull/15580)
test/recipes/80-test_cmp_http.t