Module Name:    src
Committed By:   yamaguchi
Date:           Tue May 11 01:15:11 UTC 2021

Modified Files:
        src/sys/net: if_sppp.h if_spppsubr.c

Log Message:
Revert previous commit because of mistake of commit log

back to r1.230(if_spppsubr.c) and r1.31(if_sppp.h)


To generate a diff of this commit:
cvs rdiff -u -r1.32 -r1.33 src/sys/net/if_sppp.h
cvs rdiff -u -r1.231 -r1.232 src/sys/net/if_spppsubr.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: src/sys/net/if_sppp.h
diff -u src/sys/net/if_sppp.h:1.32 src/sys/net/if_sppp.h:1.33
--- src/sys/net/if_sppp.h:1.32	Tue May 11 01:00:49 2021
+++ src/sys/net/if_sppp.h	Tue May 11 01:15:11 2021
@@ -1,4 +1,4 @@
-/*	$NetBSD: if_sppp.h,v 1.32 2021/05/11 01:00:49 yamaguchi Exp $	*/
+/*	$NetBSD: if_sppp.h,v 1.33 2021/05/11 01:15:11 yamaguchi Exp $	*/
 
 /*-
  * Copyright (c) 2002 The NetBSD Foundation, Inc.
@@ -233,15 +233,5 @@ struct spppipv6cpstatus {
 
 #define SPPPGETIPV6CPSTATUS	_IOWR('i', 137, struct spppipv6cpstatus)
 
-#define SPPP_NCP_IPCP		__BIT(0)
-#define SPPP_NCP_IPV6CP		__BIT(1)
-struct spppncpcfg {
-	char		ifname[IFNAMSIZ];
-	u_int		ncp_flags;
-};
-
-#define SPPPGETNCPCFG		_IOWR('i', 138, struct spppncpcfg)
-#define SPPPSETNCPCFG		_IOW('i', 139, struct spppncpcfg)
-
 #endif /* !_NET_IF_SPPP_H_ */
 

Index: src/sys/net/if_spppsubr.c
diff -u src/sys/net/if_spppsubr.c:1.231 src/sys/net/if_spppsubr.c:1.232
--- src/sys/net/if_spppsubr.c:1.231	Tue May 11 01:00:49 2021
+++ src/sys/net/if_spppsubr.c	Tue May 11 01:15:11 2021
@@ -1,4 +1,4 @@
-/*	$NetBSD: if_spppsubr.c,v 1.231 2021/05/11 01:00:49 yamaguchi Exp $	 */
+/*	$NetBSD: if_spppsubr.c,v 1.232 2021/05/11 01:15:11 yamaguchi Exp $	 */
 
 /*
  * Synchronous PPP/Cisco link level subroutines.
@@ -41,7 +41,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: if_spppsubr.c,v 1.231 2021/05/11 01:00:49 yamaguchi Exp $");
+__KERNEL_RCSID(0, "$NetBSD: if_spppsubr.c,v 1.232 2021/05/11 01:15:11 yamaguchi Exp $");
 
 #if defined(_KERNEL_OPT)
 #include "opt_inet.h"
@@ -585,7 +585,11 @@ sppp_input(struct ifnet *ifp, struct mbu
 			log(LOG_DEBUG,
 			    "%s: input packet is too small, %d bytes\n",
 			    ifp->if_xname, m->m_pkthdr.len);
-		goto drop;
+	  drop:
+		if_statadd2(ifp, if_ierrors, 1, if_iqdrops, 1);
+		m_freem(m);
+		SPPP_UNLOCK(sp);
+		return;
 	}
 
 	if (sp->pp_flags & PP_NOFRAMING) {
@@ -659,7 +663,7 @@ sppp_input(struct ifnet *ifp, struct mbu
 	}
 
 	switch (protocol) {
-	reject_protocol:
+	default:
 		if (sp->scp[IDX_LCP].state == STATE_OPENED) {
 			uint16_t prot = htons(protocol);
 
@@ -675,8 +679,6 @@ sppp_input(struct ifnet *ifp, struct mbu
 			    "<proto=0x%x>\n", ifp->if_xname, ntohs(protocol));
 		if_statinc(ifp, if_noproto);
 		goto drop;
-	default:
-		goto reject_protocol;
 	case PPP_LCP:
 		SPPP_UNLOCK(sp);
 		sppp_cp_input(&lcp, sp, m);
@@ -698,8 +700,6 @@ sppp_input(struct ifnet *ifp, struct mbu
 		return;
 #ifdef INET
 	case PPP_IPCP:
-		if (!ISSET(sp->pp_ncpflags, SPPP_NCP_IPCP))
-			goto reject_protocol;
 		SPPP_UNLOCK(sp);
 		if (sp->pp_phase == SPPP_PHASE_NETWORK) {
 			sppp_cp_input(&ipcp, sp, m);
@@ -717,8 +717,6 @@ sppp_input(struct ifnet *ifp, struct mbu
 #endif
 #ifdef INET6
 	case PPP_IPV6CP:
-		if (!ISSET(sp->pp_ncpflags, SPPP_NCP_IPV6CP))
-			goto reject_protocol;
 		SPPP_UNLOCK(sp);
 		if (sp->pp_phase == SPPP_PHASE_NETWORK) {
 			sppp_cp_input(&ipv6cp, sp, m);
@@ -768,13 +766,6 @@ queue_pkt:
 	IF_ENQUEUE(inq, m);
 	IFQ_UNLOCK(inq);
 	schednetisr(isr);
-	return;
-
-drop:
-	if_statadd2(ifp, if_ierrors, 1, if_iqdrops, 1);
-	m_freem(m);
-	SPPP_UNLOCK(sp);
-	return;
 }
 
 /*
@@ -1038,7 +1029,6 @@ sppp_attach(struct ifnet *ifp)
 	sp->pp_phase = SPPP_PHASE_DEAD;
 	sp->pp_up = sppp_notify_up;
 	sp->pp_down = sppp_notify_down;
-	sp->pp_ncpflags = SPPP_NCP_IPCP | SPPP_NCP_IPV6CP;
 	sppp_wq_set(&sp->work_ifdown, sppp_ifdown, NULL);
 	memset(sp->scp, 0, sizeof(sp->scp));
 	rw_init(&sp->pp_lock);
@@ -1275,7 +1265,6 @@ sppp_ioctl(struct ifnet *ifp, u_long cmd
 
 	case SPPPSETAUTHCFG:
 	case SPPPSETLCPCFG:
-	case SPPPSETNCPCFG:
 	case SPPPSETIDLETO:
 	case SPPPSETAUTHFAILURE:
 	case SPPPSETDNSOPTS:
@@ -1295,7 +1284,6 @@ sppp_ioctl(struct ifnet *ifp, u_long cmd
 
 	case SPPPGETAUTHCFG:
 	case SPPPGETLCPCFG:
-	case SPPPGETNCPCFG:
 	case SPPPGETAUTHFAILURES:
 		error = kauth_authorize_network(l->l_cred,
 		    KAUTH_NETWORK_INTERFACE,
@@ -3503,9 +3491,6 @@ sppp_ipcp_open(struct sppp *sp, void *xc
 	KASSERT(SPPP_WLOCKED(sp));
 	KASSERT(!cpu_softintr_p());
 
-	if (!ISSET(sp->pp_ncpflags, SPPP_NCP_IPCP))
-		return;
-
 	sp->ipcp.flags &= ~(IPCP_HISADDR_SEEN|IPCP_MYADDR_SEEN|IPCP_MYADDR_DYN|IPCP_HISADDR_DYN);
 	sp->ipcp.req_myaddr = 0;
 	sp->ipcp.req_hisaddr = 0;
@@ -4080,9 +4065,6 @@ sppp_ipv6cp_open(struct sppp *sp, void *
 	KASSERT(SPPP_WLOCKED(sp));
 	KASSERT(!cpu_softintr_p());
 
-	if (!ISSET(sp->pp_ncpflags, SPPP_NCP_IPV6CP))
-		return;
-
 #ifdef IPV6CP_MYIFID_DYN
 	sp->ipv6cp.flags &= ~(IPV6CP_MYIFID_SEEN|IPV6CP_MYIFID_DYN);
 #else
@@ -6131,24 +6113,6 @@ sppp_params(struct sppp *sp, u_long cmd,
 		SPPP_UNLOCK(sp);
 	    }
 	    break;
-	case SPPPGETNCPCFG:
-	    {
-		struct spppncpcfg *ncpp = (struct spppncpcfg *) data;
-
-		SPPP_LOCK(sp, RW_READER);
-		ncpp->ncp_flags = sp->pp_ncpflags;
-		SPPP_UNLOCK(sp);
-	    }
-		break;
-	case SPPPSETNCPCFG:
-	    {
-		struct spppncpcfg *ncpp = (struct spppncpcfg *) data;
-
-		SPPP_LOCK(sp, RW_WRITER);
-		sp->pp_ncpflags = ncpp->ncp_flags;
-		SPPP_UNLOCK(sp);
-	    }
-		break;
 	case SPPPGETSTATUS:
 	    {
 		struct spppstatus *status = (struct spppstatus *)data;

Reply via email to