abc/lwip
1
0
Fork 0
mirror of https://git.savannah.nongnu.org/git/lwip.git synced 2025-08-03 21:14:40 +08:00
Code Issues Packages Projects Releases Wiki Activity

PPP, explain better why we don't need protp->datainput callback

Browse Source
This commit is contained in:
Sylvain Rochet 2015-08-27 22:42:42 +02:00
parent 36b3878a45
commit 46985bf750
1 changed files with 8 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
src/netif/ppp/ppp.c
View File

@ -878,12 +878,16 @@ void ppp_input(ppp_pcb *pcb, struct pbuf *pb) {
} }
#if 0 /* UNUSED #if 0 /* UNUSED
* *
* This is actually a (hacked?) way for the PPP kernel implementation to pass a * This is actually a (hacked?) way for the Linux kernel to pass a data
* data packet to the PPP daemon. The PPP daemon normally only do signaling * packet to pppd. pppd in normal condition only do signaling
* (LCP, PAP, CHAP, IPCP, ...) and does not handle any data packet at all. * (LCP, PAP, CHAP, IPCP, ...) and does not handle any data packet at all.
* *
* This is only used by CCP, which we cannot support until we have a CCP data * We don't even need this interface, which is only there because of PPP
* implementation. * interface limitation between Linux kernel and pppd. For MPPE, which uses
* CCP to negotiate although it is not really a (de)compressor, we added
* ccp_resetrequest() in CCP and MPPE input data flow is calling either
* ccp_resetrequest() or lcp_close() if the issue is, respectively, non-fatal
* or fatal, this is what ccp_datainput() really do.
*/ */
if (protocol == (protp->protocol & ~0x8000) if (protocol == (protp->protocol & ~0x8000)
&& protp->datainput != NULL) { && protp->datainput != NULL) {