Fix mem leak in sslapi test
authorMatt Caswell <matt@openssl.org>
Wed, 9 May 2018 09:45:46 +0000 (10:45 +0100)
committerMatt Caswell <matt@openssl.org>
Fri, 11 May 2018 13:51:08 +0000 (14:51 +0100)
commitc20e3b282c26205f39a89a23664245475d4d7cbc
tree874ba9eab6060d17acb67bbeea1adb83241ca8f4
parentd0191fe059fc73ff70af31de1fb99f2d2298242c
Fix mem leak in sslapi test

The recent change in behaviour where you do not get a NewSessionTicket
message sent if you established the connection using a PSK caused a mem
leak to be triggered in sslapitest. It was actually a latent bug and we
were just lucky we never hit it before. The problem is due to complexity
with the way PSK sessions were set up in the early_data tests. PSK session
reference counting was handled differently to normal session reference
counting. This meant there were lots of special cases in the code where
we don't free a session if it is a PSK. It makes things easier if we just
handle PSK reference counts in the same way as other session reference
counts, and then we can remove all of the special case code.

Reviewed-by: Viktor Dukhovni <viktor@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/6198)
test/sslapitest.c