diff options
author | brian <brian@FreeBSD.org> | 2000-11-07 04:29:33 +0000 |
---|---|---|
committer | brian <brian@FreeBSD.org> | 2000-11-07 04:29:33 +0000 |
commit | a38b1bf086bcd8a4ea1ab6baa4cc24633dfdda9d (patch) | |
tree | 6ad87a737721df4fd1529d04cd95d58e9025ee6d /usr.sbin/ppp/ccp.c | |
parent | e30917a73ea8d933eeeac4fd502c38b5b81cd052 (diff) | |
download | FreeBSD-src-a38b1bf086bcd8a4ea1ab6baa4cc24633dfdda9d.zip FreeBSD-src-a38b1bf086bcd8a4ea1ab6baa4cc24633dfdda9d.tar.gz |
Pass the correct output options to the ccp output initialisation
routine rather than passing it the first requested output option.
Ensure that options are freed correctly even if we don't reach
TLU.
Diffstat (limited to 'usr.sbin/ppp/ccp.c')
-rw-r--r-- | usr.sbin/ppp/ccp.c | 23 |
1 files changed, 21 insertions, 2 deletions
diff --git a/usr.sbin/ppp/ccp.c b/usr.sbin/ppp/ccp.c index f3df405..01fbd69 100644 --- a/usr.sbin/ppp/ccp.c +++ b/usr.sbin/ppp/ccp.c @@ -376,7 +376,20 @@ static void CcpLayerFinish(struct fsm *fp) { /* We're now down */ + struct ccp *ccp = fsm2ccp(fp); + struct ccp_opt *next; + log_Printf(LogCCP, "%s: LayerFinish.\n", fp->link->name); + + /* + * Nuke options that may be left over from sending a REQ but never + * coming up. + */ + while (ccp->out.opt) { + next = ccp->out.opt->next; + free(ccp->out.opt); + ccp->out.opt = next; + } } /* Called when CCP has reached the OPEN state */ @@ -385,6 +398,8 @@ CcpLayerUp(struct fsm *fp) { /* We're now up */ struct ccp *ccp = fsm2ccp(fp); + struct ccp_opt **o; + int f; log_Printf(LogCCP, "%s: LayerUp.\n", fp->link->name); @@ -400,10 +415,14 @@ CcpLayerUp(struct fsm *fp) } } + o = &ccp->out.opt; + for (f = 0; f < ccp->out.algorithm; f++) + if (IsEnabled(ccp->cfg.neg[algorithm[f]->Neg])) + o = &(*o)->next; + if (ccp->out.state == NULL && ccp->out.algorithm >= 0 && ccp->out.algorithm < NALGORITHMS) { - ccp->out.state = (*algorithm[ccp->out.algorithm]->o.Init) - (&ccp->out.opt->val); + ccp->out.state = (*algorithm[ccp->out.algorithm]->o.Init)(&(*o)->val); if (ccp->out.state == NULL) { log_Printf(LogERROR, "%s: %s (out) initialisation failure\n", fp->link->name, protoname(ccp->my_proto)); |