[lkml]   [2001]   [May]   [24]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: SyncPPP IPCP/LCP loop problem and patch
Paul Fulghum wrote:
> RFC1661 state table shows a transition to req-sent
> from opened when a (properly formated with
> correct sequence ID) cfg-ack is received.
> Syncppp does not do this (from sppp_lcp_input):
> Maybe adding:
> sppp_lcp_open (sp);
> sp->ipcp.state = IPCP_STATE_CLOSED;
> sp->lcp.state = LCP_STATE_REQ_SENT;
> break;

Thanks for the suggestion. I tried it and found that Linux syncppp does not
have a LCP_STATE_REQ_SENT the nearest alternate being LCP_STATE_CLOSED.
Looking at the RFC these are by no means compatable. Adding the extra state
and coding the transitions would not be difficult but as I've looked at it
I've seen more and more ommissions in the code. The magic number is not
randomised very well, jiffies is not a great random number :-) Parameter
negotiation only takes place for the magic number plus dummys for MRU and
ACCM, I'm not even sure that enforcing and ACCM of all zeros is required.
If the remote end sends us a config request without a magic number we end
up testing an uninitialised variable, and so on.

Who's the owner for syncppp.c ? I might be able to put some time in on
it, but would hate to be sending patches into empty space.

Bob Dunlop FarSite Communications
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to
More majordomo info at
Please read the FAQ at

 \ /
  Last update: 2005-03-22 12:53    [W:0.073 / U:0.408 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site