Bug#538107: multilink with nodetach [Fixed]

2014-01-06 Thread Stephen Marron
Hi,

This bug has been resolved in upstream git, and the version in jessie
2.4.5+git20130610-1 includes the fix.

Regards,

Stephen

commit 398ed2585640d198c53e736ee5bbd67f7ce8168e
Author: Paul Mackerras pau...@samba.org
Date:   Sat Mar 2 20:25:28 2013 +1100

pppd: Add master_detach option

This adds a new option, master_detach, to allow pppd to detach from
the controlling terminal when it is the multilink bundle master but
its own link has terminated, even if the nodetach option has been
given.

Requested-by: Stephen Marron s...@boxfusion.net
Signed-off-by: Paul Mackerras pau...@samba.org


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#538107:

2010-08-24 Thread Stephen Marron
Hi,

The below diff is against upstream git repo

git://git.ozlabs.org/~paulus/ppp.git

on

Tue, 24 Aug 2010 16:44:43 +0930

this patch should be used rather than the previous one

diff --git a/pppd/auth.c b/pppd/auth.c
index fb71944..62648b8 100644
--- a/pppd/auth.c
+++ b/pppd/auth.c
@@ -669,8 +669,11 @@ link_terminated(unit)
(*the_channel-cleanup)();

 if (doing_multilink  multilink_master) {
-   if (!bundle_terminating)
+   if (!bundle_terminating) {
new_phase(PHASE_MASTER);
+   if (!detached)
+   detach();
+   }
else
mp_bundle_terminated();
 } else



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#538107: fixed patch

2010-08-24 Thread Stephen Marron
Hi,

The below diff is against upstream git repo

git://git.ozlabs.org/~paulus/ppp.git

on

Tue, 24 Aug 2010 16:44:43 +0930

this patch should be used rather than the previous one

diff --git a/pppd/auth.c b/pppd/auth.c
index fb71944..62648b8 100644
--- a/pppd/auth.c
+++ b/pppd/auth.c
@@ -669,8 +669,11 @@ link_terminated(unit)
(*the_channel-cleanup)();

 if (doing_multilink  multilink_master) {
-   if (!bundle_terminating)
+   if (!bundle_terminating) {
new_phase(PHASE_MASTER);
+   if (!detached)
+   detach();
+   }
else
mp_bundle_terminated();
 } else



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#538107: multilink with nodetach [PATCH]

2010-08-24 Thread Stephen Marron
Hi

This bug has been Reported in the Debian BTS as bug 538107

http://bugs.debian.org/578070

If pppd is used with l2tpd (nodetach) in a multilink situation then this
feature of pppd causes problems

MULTILINK
If the first link terminates (for example, because of  a
hangup or  a received  LCP  terminate-request)  the  bundle  is
not destroyed unless there are no other links remaining in the
bundle.  Rather than exiting, the first pppd keeps running
after its link terminates, until all the links in the bundle
have terminated.

pppd remains connected to l2tpd even though it is just acting as a
multilink master, causing l2tpd to believe that pppd is connected... i
have resolved the issue in my own system by instructing pppd to detach
from the terminal after it changes to PHASE_MASTER i have been using
this patch since 2009-02-19 in our production systems and havent had any
problems

The if (!detached) call is redundant since the detach function already
checks if the process is detached so can be optionally excluded

I have noticed other people complaining of similar issues when using
multilink/nodetach in other setups (eg pppoe) I believe that this patch
may also solve the problem for them, but i have not tested that setup

-
diff --git a/pppd/auth.c b/pppd/auth.c
index fb71944..62648b8 100644
--- a/pppd/auth.c
+++ b/pppd/auth.c
@@ -669,8 +669,11 @@ link_terminated(unit)
(*the_channel-cleanup)();

 if (doing_multilink  multilink_master) {
-   if (!bundle_terminating)
+   if (!bundle_terminating) {
new_phase(PHASE_MASTER);
+   if (!detached)
+   detach();
+   }
else
mp_bundle_terminated();
 } else
-

feedback is appreciated :)

Regards

Stephen




-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#538107: fixed patch

2010-08-24 Thread Marco d'Itri
On Aug 24, Stephen Marron deb...@solo.boxfusion.net wrote:

 The below diff is against upstream git repo
I do not feel comfortable applying at this time a patch to code I do not
understand, can you send it to the upstream mailing list and have
somebody say it's OK?

-- 
ciao,
Marco


signature.asc
Description: Digital signature


Bug#538107: Acknowledgement (multilink with nodetach )

2009-10-06 Thread SoloFlyer

Hi,

Is there any chance that this patch will make it in before squeeze gets 
frozen in december?


Thanks

Stephen




--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#538107: multilink with nodetach

2009-07-22 Thread SoloFlyer

Package: ppp
Version: 2.4.4rel-10.1
Severity: minor 

If pppd is used with l2tpd (nodetach) in a multilink situation then this 
feature of pppd causes problems


  MULTILINK
  If the first link terminates (for example, because of  a  hangup  
or  a
  received  LCP  terminate-request)  the  bundle  is not destroyed 
unless
  there are no other links remaining in the bundle.  Rather than 
exiting,
  the  first  pppd keeps running after its link terminates, until 
all the

  links in the bundle have terminated.

pppd remains connected to l2tpd even though it is just acting as a 
multilink master, causing l2tpd to belive the pppd is connected... i 
have resolved the issue in my own system by instructing pppd to detach 
from the terminal after it changes to PHASE_MASTER i have been using 
this patch since 2009-02-19 in our production systems and havent had any 
problems


The if (!detached) call is redundant since the detach function already 
checks if the process is detached so can be optionally excluded


i have noticed other people complaining of similar issues when using 
multilink/nodetach in other setups (eg pppoe) I believe that this patch 
may also solve the problem for them, but i have not tested that setup


-
(auth.c)@(void link_terminated(unit))

  if (doing_multilink  multilink_master) {
   if (!bundle_terminating)
   new_phase(PHASE_MASTER);
+++if (!detached)
+++detach();
   else
   mp_bundle_terminated();
   } else
   new_phase(PHASE_DEAD);
-

feedback is appreciated :)

Regards

Stephen




--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org