Hello,
Is it OK to merge the PRU_CONNECT2 case into the generic
"not supported" case in {rip,rip6}_usrreq()?
If anything this allows the reader to see all the
unsupported requests in one place.
- Michael
Index: netinet/raw_ip.c
===================================================================
RCS file: /cvs/src/sys/netinet/raw_ip.c,v
retrieving revision 1.119
diff -u -p -u -r1.119 raw_ip.c
--- netinet/raw_ip.c 4 Feb 2019 21:40:52 -0000 1.119
+++ netinet/raw_ip.c 21 Feb 2019 07:15:08 -0000
@@ -493,10 +493,6 @@ rip_usrreq(struct socket *so, int req, s
break;
}
- case PRU_CONNECT2:
- error = EOPNOTSUPP;
- break;
-
/*
* Mark the connection as being incapable of further input.
*/
@@ -549,6 +545,7 @@ rip_usrreq(struct socket *so, int req, s
/*
* Not supported.
*/
+ case PRU_CONNECT2:
case PRU_LISTEN:
case PRU_ACCEPT:
case PRU_SENDOOB:
Index: netinet6/raw_ip6.c
===================================================================
RCS file: /cvs/src/sys/netinet6/raw_ip6.c,v
retrieving revision 1.134
diff -u -p -u -r1.134 raw_ip6.c
--- netinet6/raw_ip6.c 4 Feb 2019 21:40:52 -0000 1.134
+++ netinet6/raw_ip6.c 21 Feb 2019 07:15:09 -0000
@@ -618,10 +618,6 @@ rip6_usrreq(struct socket *so, int req,
break;
}
- case PRU_CONNECT2:
- error = EOPNOTSUPP;
- break;
-
/*
* Mark the connection as being incapable of futther input.
*/
@@ -672,6 +668,7 @@ rip6_usrreq(struct socket *so, int req,
/*
* Not supported.
*/
+ case PRU_CONNECT2:
case PRU_LISTEN:
case PRU_ACCEPT:
case PRU_SENDOOB:
