Fix bug in dpdk_crypto_session_disposal() 02/10802/2
authorMatthew Smith <mgsmith@netgate.com>
Sun, 25 Feb 2018 02:51:53 +0000 (20:51 -0600)
committerDamjan Marion <dmarion.lists@gmail.com>
Sun, 25 Feb 2018 22:04:27 +0000 (22:04 +0000)
commit3e57d78a9ad65c6b2a9b68e7bb52b6902f574152
tree69e033832e06ed3f5757563c2e807812516b789b
parent329a1c10b7120f0be75aa0e0187fbc9f87e760f9
Fix bug in dpdk_crypto_session_disposal()

The expression to determine whether to delete a session
from the disposal list only evaluates true if some,
but not all, of the sessions in the list were freed.
When all sessions in the list are freed, it evaluates
false and the sessions are left in the list to be freed
again later, which can result in a session pool element
that was reallocated to a different SA being freed,
breaking crypto for the newer SA.

Add an 'else' that handles the case where all sessions
were freed.

Change-Id: I3ae54d5b3bfc3658bf406caa50646924baaae589
Signed-off-by: Matthew Smith <mgsmith@netgate.com>
src/plugins/dpdk/ipsec/ipsec.c