Browse Source

Check for remote IP duplicates in pptpd.conf

Christoph Biedl 7 years ago
parent
commit
f7e1107415

+ 33 - 0
debian/patches/cherry-pick.1.4.0-16-g0c0ad67.check-for-remote-ip-duplicates-in-pptpd-conf.patch

@@ -0,0 +1,33 @@
+Subject: Check for remote IP duplicates in pptpd.conf
+Origin: 1.4.0-16-g0c0ad67
+Upstream-Author: Christoph Biedl <sourceforge.bnwi@manchmal.in-ulm.de>
+Date: Fri Jul 8 13:51:42 2016 +1000
+
+    At the moment, pptpd will happily process lines like
+    
+    	remoteip 192.168.0.234-234,192.168.0.234,192.168.0.234
+    
+    and afterwards state "MGR: Maximum of 3 connections available". While
+    technically true, I guess this is never by intention but will result in
+    interesting behaviour.  [...]
+    
+    The patch below worked for me.
+    
+    Signed-off-by: James Cameron <quozl@laptop.org>
+
+--- a/pptpmanager.c
++++ b/pptpmanager.c
+@@ -136,6 +136,13 @@
+ void slot_set_remote(int i, char *ip)
+ {
+   struct slot *slot = &slots[i];
++  int j;
++  for (j = 0; j < slot_count; j++) {
++    if (!slots[j].remote || strcmp(slots[j].remote, ip))
++      continue;
++    syslog(LOG_ERR, "MGR: Remote IP address %s in config is a duplicate!", ip);
++    exit(1);
++  }
+   if (slot->remote) free(slot->remote);
+   slot->remote = strdup(ip);
+ }

+ 1 - 0
debian/patches/series

@@ -2,6 +2,7 @@
 cherry-pick.1.4.0-10-g6adffc1.bcrelay-fix-typo.patch
 cherry-pick.1.4.0-10-g6adffc1.bcrelay-fix-typo.patch
 cherry-pick.1.4.0-11-g4ea2db6.ff.bcrelay-c-replace-sprintf-with-snprintf.patch
 cherry-pick.1.4.0-11-g4ea2db6.ff.bcrelay-c-replace-sprintf-with-snprintf.patch
 cherry-pick.1.4.0-14-gb84360b.pptpctrl-exit-when-bailing.patch
 cherry-pick.1.4.0-14-gb84360b.pptpctrl-exit-when-bailing.patch
+cherry-pick.1.4.0-16-g0c0ad67.check-for-remote-ip-duplicates-in-pptpd-conf.patch
 
 
 # local modifications
 # local modifications
 local.build_hardening-flags.patch
 local.build_hardening-flags.patch