Bug 1383845

Summary: dpdaction=restart causing a delay during which packets leak plaintext
Product: Red Hat Enterprise Linux 7 Reporter: Paul Wouters <pwouters>
Component: libreswanAssignee: Paul Wouters <pwouters>
Status: CLOSED WONTFIX QA Contact: BaseOS QE Security Team <qe-baseos-security>
Severity: medium Docs Contact:
Priority: medium    
Version: 7.4CC: mgrepl, omoris, pwouters
Target Milestone: rc   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
: 1383846 (view as bug list) Environment:
Last Closed: 2018-12-20 17:30:26 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Bug Depends On:    
Bug Blocks: 1383846    

Description Paul Wouters 2016-10-12 00:11:03 UTC 
This action causes a delete + add + initiate, the leak happens between delete and initiate. Apparently this can be 30s

A workaround is to use dpdaction=hold instead of dpdaction=restart

the hold action prevents the delete + add and puts the existing connection/state into initiating mode.
Comment 5 Ondrej Moriš 2018-08-13 14:35:52 UTC 
Paul, isn't this included in 3.25 rebase?
Comment 6 Paul Wouters 2018-08-13 18:13:27 UTC 
No this issue has not been addressed yet. We still recommend not using the dpdaction=restart.

The idea is that a connection with auto=start or that was ipsec auto --up'ed is in a state where it will re-initiate if something happens, such as DPD causing the connection to go into "hold". That is, another mechanism triggers a new initialization of the connection, not dpd code directly.
Comment 7 Paul Wouters 2018-12-20 17:30:26 UTC 
This bug was not fixed in time for RHEL7 and has been moved to RHEL8