Project

General

Profile

Actions

Bug #6263

closed

Encryption options for every P2 on a given P1 are written to each P2 individually inside ipsec.conf with multiple P2 entries + split conn entries

Added by Jim Pingle almost 8 years ago. Updated over 4 years ago.

Status:
Resolved
Priority:
Normal
Category:
IPsec
Target version:
Start date:
04/25/2016
Due date:
% Done:

100%

Estimated time:
Plus Target Version:
Release Notes:
Affected Version:
All
Affected Architecture:

Description

When multiple Phase 2 entries exist, the encryption options for every P2 inside a given P1 are added to every split connection entries (IKEv1 or IKEv2 with split connections active)

For example, with three P2 entries all set for AES256-GCM-128, ipsec.conf has this in each conn entry:

    esp = aes256gcm128,aes256gcm128,aes256gcm128!

The same identical option is written on each connection.

If different options are used for P2s within the same tunnel, all of them are written to each P2. So each split conn entry contains all possible P2 algorithms for any P2 on the P1, not just the selected algorithm for that specific P2 entry.


Files

pfsense-bug-6263.diff (528 Bytes) pfsense-bug-6263.diff PJ Goodwin, 02/05/2018 05:41 PM
Actions #1

Updated by Chris Buechler almost 8 years ago

  • Target version changed from 2.3.1 to 2.3.2
  • Affected Version changed from 2.3 to All

always been that way (well, w/strongswan, >=2.2.0). Doesn't hurt anything, not worth risking at this stage for 2.3.1.

Actions #2

Updated by Chris Buechler almost 8 years ago

  • Target version changed from 2.3.2 to 2.4.0
Actions #3

Updated by si lec over 7 years ago

might be always been that way but this is very painful... all other brand do support this properly.
I just start using pfSense and this bug cause me quite a lot of trouble. You really need to fix this

Actions #4

Updated by Jim Thompson over 7 years ago

  • Assignee set to Matthew Smith
Actions #5

Updated by Jim Thompson over 6 years ago

  • Assignee changed from Matthew Smith to Renato Botelho
Actions #6

Updated by Cullen Trey over 6 years ago

Hi,

we had problems with one of our IPSecs. Sometimes it can connect, sometimes not. After a reboot of pfsense, pfsense can mostly connect.

When I read about this bug, I disabled all P2 entries except one. Now it always works...

Could it really be, that the entry with the same encryption multiple times (esp = aes256gcm128,aes256gcm128,aes256gcm128!) is making problems with the remote site? In our setup, only we establish the ipsec connection.

We be greate to see this fixed, as it can not do anything good.

thanks for your work!

Actions #7

Updated by Renato Botelho over 6 years ago

  • Target version changed from 2.4.0 to 2.4.1
Actions #8

Updated by Jim Pingle over 6 years ago

  • Target version changed from 2.4.1 to 2.4.2
Actions #9

Updated by Jim Pingle over 6 years ago

  • Target version changed from 2.4.2 to 2.4.3
Actions #10

Updated by Jim Pingle over 6 years ago

  • Target version changed from 2.4.3 to 2.4.4
Actions #11

Updated by PJ Goodwin about 6 years ago

Ran into this bug as well, though it appears to break things if you have too many phase 2 entries. After a certain number (currently appears to be around 85 or 90) phase 2 networks, things start becoming flakey. Sometimes the phase 2 entries at the bottom will work, and other times they don't.

If I modify the /var/etc/ipsec.conf file to make the esp config entries just have the single entry for the last few phase 2 sections, all the networks work / are stable.

If you need any additional info on our setup, let me know.

If someone can point me in the direction of what file creates the ipsec.conf file, I can take a look at it and see if I can figure out a fix. I'm going to try to get time to search for the file myself, but not sure when I'll be able to take a look.

thanks.

Actions #12

Updated by PJ Goodwin about 6 years ago

Looked into this and the attached patch appears to fix the issue in 2.4.2. The comparable change also corrected a 2.3.1 version.

It simply resets the $ealgoESPsp2arr variable for each phase2 entry.

I didn't look into it but it's possible that $ealgoAHsp2arr may need to be reset at the appropriate point as well. I don't have a setup that uses that so I didn't look into it.

Actions #13

Updated by PJ Goodwin over 5 years ago

PJ Goodwin wrote:

Looked into this and the attached patch appears to fix the issue in 2.4.2. The comparable change also corrected a 2.3.1 version.

It simply resets the $ealgoESPsp2arr variable for each phase2 entry.

I didn't look into it but it's possible that $ealgoAHsp2arr may need to be reset at the appropriate point as well. I don't have a setup that uses that so I didn't look into it.

Same fix corrected the issue on version 2.4.3, though the section moved to line 1264.

pj.

Actions #14

Updated by Jim Pingle over 5 years ago

  • Target version changed from 2.4.4 to 48
Actions #15

Updated by Jim Pingle about 5 years ago

  • Target version changed from 48 to 2.5.0
Actions #16

Updated by Renato Botelho over 4 years ago

  • Status changed from Confirmed to Feedback
  • % Done changed from 0 to 100
Actions #17

Updated by Jim Pingle over 4 years ago

  • Status changed from Feedback to Resolved

This looks good to me, duplicate items are no longer present.

Actions #18

Updated by Christoph Krapp over 4 years ago

Nice to see that this issue got resolved.
Nevertheless, I'm wondering if the changes could be backported to the branches RELENG_2_4_5 & RELENG_2_4_4? The changeset was only applied to master but those changes got reworked in the context of #9603. It would be really nice to see this patched in one of the next releases, otherwise people have to wait for 2.5/3.0.
Thanks in advance for your effort.

Actions #19

Updated by Renato Botelho over 4 years ago

  • Status changed from Resolved to Feedback
  • Target version changed from 2.5.0 to 2.4.5

I've cherry-picked it to 2.4.5

Actions #20

Updated by Viktor Gurov over 4 years ago

  • Status changed from Feedback to Resolved

Renato Botelho wrote:

I've cherry-picked it to 2.4.5

tested on 2.4.5.a.20200107.1903
works as expected

Actions

Also available in: Atom PDF