Package: pptpd
Version: 1.3.0-1

Could you please consider patching the current PoPToP package (pptpd) for Debian testing/unstable, version 1.3.0-1 using the attached patch? It is a backport patch from the current experimental version 1.3.2 (see: http://www.poptop.org/) which disables the DEBUG logging to the syslog when the debug option has not been turned on in the /etc/pptpd.conf file.

The current version makes my syslog grow really fast because it logs many messages like this: "GRE: accepting packet 1234".

--- pptpd-1.3.0/pptpgre.c       2005-08-02 13:33:31.000000000 +0200
+++ pptpd-1.3.0-patched/pptpgre.c       2006-06-26 14:06:12.000000000 +0200
@@ -37,6 +37,7 @@
 #include "ppphdlc.h"
 #include "pptpgre.h"
 #include "pptpdefs.h"
+#include "pptpctrl.h"
 #include "defaults.h"
 #include "pqueue.h"
 
@@ -317,9 +318,13 @@
                /* if it is timed out... */
                if (head->seq != gre.seq_recv + 1 ) {  /* wrap-around safe */
                        stats.rx_lost += head->seq - gre.seq_recv - 1;
-                       syslog(LOG_DEBUG, "GRE: timeout waiting for %d 
packets", head->seq - gre.seq_recv - 1);        
+                       if (pptpctrl_debug) {
+                               syslog(LOG_DEBUG, "GRE: timeout waiting for %d 
packets", head->seq - gre.seq_recv - 1);
+                       }
+               }
+               if (pptpctrl_debug) {
+                       syslog(LOG_DEBUG, "GRE: accepting #%d from queue", 
head->seq);
                }
-               syslog(LOG_DEBUG, "GRE: accepting #%d from queue", head->seq);
                gre.seq_recv = head->seq;
                status = callback(cl, head->packet, head->packlen);
                pqueue_del(head);
@@ -399,16 +404,22 @@
                }
                /* check for out-of-order sequence number */
                if (seq_greater(seq, gre.seq_recv)) {
-                       syslog(LOG_DEBUG, "GRE: accepting packet #%d", seq);
+                       if (pptpctrl_debug) {
+                               syslog(LOG_DEBUG, "GRE: accepting packet #%d", 
seq);
+                       }
                        stats.rx_accepted++;
                        gre.seq_recv = seq;
                        return cb(cl, buffer + ip_len + headersize, 
payload_len);
                } else if (seq == gre.seq_recv) {
-                       syslog(LOG_DEBUG, "GRE: discarding duplicate or old 
packet #%d (expecting #%d)", seq, gre.seq_recv + 1);
+                       if (pptpctrl_debug) {
+                               syslog(LOG_DEBUG, "GRE: discarding duplicate or 
old packet #%d (expecting #%d)", seq, gre.seq_recv + 1);
+                       }
                        return 0;       /* discard duplicate packets */
                } else {
                        stats.rx_buffered++;
-                       syslog(LOG_DEBUG, "GRE: buffering packet #%d (expecting 
#%d, lost or reordered)", seq, gre.seq_recv + 1);
+                       if (pptpctrl_debug) {
+                               syslog(LOG_DEBUG, "GRE: buffering packet #%d 
(expecting #%d, lost or reordered)", seq, gre.seq_recv + 1);
+                       }
                        pqueue_add(seq, buffer + ip_len + headersize, 
payload_len);
                        return 0;       /* discard out-of-order packets */
                }

Reply via email to