Bug 152101 - Updating from 4.6.4 to 4.7 breaks "pptp pass through"
Summary: Updating from 4.6.4 to 4.7 breaks "pptp pass through"
Status: NEW
Alias: None
Product: Networking
Classification: Unclassified
Component: IPV4 (show other bugs)
Hardware: All Linux
: P1 normal
Assignee: Stephen Hemminger
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-08-12 11:05 UTC by Francisco Pina Martins
Modified: 2017-02-14 18:20 UTC (History)
7 users (show)

See Also:
Kernel Version: 4.7
Subsystem:
Regression: Yes
Bisected commit-id:


Attachments
Fixes the cleanup of gre module (1.04 KB, patch)
2016-08-15 12:06 UTC, Johanna
Details | Diff

Description Francisco Pina Martins 2016-08-12 11:05:45 UTC
I have a home server running Archlinux (ITXBox), which among other things, is acting as a router for my home network, including firewall.
Up until yesterday, I was able to establish a pptp VPN connection between any machine on my network and an external server. In order for this to work, I had to add the following to ITXBox's "/etc/modules.load.d/pptp-forward.conf":

nf_conntrack_proto_gre
nf_nat_proto_gre
nf_conntrack_pptp
nf_nat_pptp

However, last night, I upgraded the kernel from 4.6.4 to the latest version in Archlinux's repositories - 4.7.
This has caused the VPN connections mentioned above to stop working.
Reverting back to 4.6.4 fixes the issue.
I did not perform a proper regression testing, since I don't currently have enough time for that (I guess it's not "silly season" for everyone), but some quick research indicated this(1) commit could have something to do with it. It was the only thing I found related to GRE traffic.
This should be easily reproducible provided the required networking infrastructure is present.


(1) https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=e0c20967c8a653d0213238621381e224d8f065fc
Comment 1 Francisco Pina Martins 2016-08-12 11:13:53 UTC
I'd just like to add that I did not find any changes to Archlinux's kernel config relevant for breaking this. You can consult these changes here:
https://git.archlinux.org/svntogit/packages.git/commit/trunk?h=packages/linux&id=c96df34bef887421b81b39598e4830d26cd1569c

There are a couple of ipv6 new modules added, but I don't think they are relevant to the issue (I could be wrong, though).
Comment 2 Richard Liu 2016-08-12 11:58:54 UTC
I used Gentoo Linux, and upgrade from 4.6.3 to 4.7. 
Also has pptp pass through issue .
Comment 3 Francisco Pina Martins 2016-08-12 14:31:20 UTC
Another possible cause for the issue is this commit(1).
Although it's only supposed to deal with GRE under ipv6. It's from the same author and committer as the previously mentioned suspect commit, so no need to add anyone else to the cc list.

(1) https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=3a80e1facd3c825c5ac804bc2efe118872832e33
Comment 4 Johanna 2016-08-15 12:06:25 UTC
Created attachment 228831 [details]
Fixes the cleanup of gre module

Would any of you experiencing this bug try the attached patch?
Comment 5 Johanna 2016-08-15 12:59:14 UTC
(In reply to Johanna from comment #4)
> Created attachment 228831 [details]
> Fixes the cleanup of gre module
> 
> Would any of you experiencing this bug try the attached patch?

sry, this patch doesn't really make sense, I misread. You don't need to try it.
Comment 6 Konstantin 2016-08-19 10:32:46 UTC
sysctl net.netfilter.nf_conntrack_helper=1
Comment 7 Francisco Pina Martins 2016-09-01 07:46:50 UTC
Thank you @Konstantin.
Confirming that adding

net.netfilter.nf_conntrack_helper=1

to /etc/sysctl.d/30-pptp_passthrough.conf solves the issue for me.
Just out of curiosity: Is this option new to 4.7 or did it exist before and the default value was changed?
Comment 8 Valentin Kulesh 2016-09-28 11:46:36 UTC
> Just out of curiosity: Is this option new to 4.7 or did it exist before and
> the default value was changed?
My testing shows that the default value has been changed.
I didn't look in the code, though -- just compared two ELRepo's kernels

Note You need to log in before you can comment on or make changes to this bug.