Add missing #ifdefs to pppx_if_destroy() as it done in
pipex_destroy_session(). Also remove unnecessary cast.

Index: sys/net/if_pppx.c
===================================================================
RCS file: /cvs/src/sys/net/if_pppx.c,v
retrieving revision 1.76
diff -u -p -r1.76 if_pppx.c
--- sys/net/if_pppx.c   20 Feb 2020 16:56:52 -0000      1.76
+++ sys/net/if_pppx.c   26 Mar 2020 10:07:26 -0000
@@ -967,13 +967,14 @@ pppx_if_destroy(struct pppx_dev *pxd, st
 
        LIST_REMOVE(session, id_chain);
        LIST_REMOVE(session, session_list);
-       switch (session->protocol) {
-       case PIPEX_PROTO_PPTP:
-       case PIPEX_PROTO_L2TP:
-               LIST_REMOVE((struct pipex_session *)session,
-                   peer_addr_chain);
-               break;
-       }
+#ifdef PIPEX_PPTP
+       if (session->protocol == PIPEX_PROTO_PPTP)
+               LIST_REMOVE(session, peer_addr_chain);
+#endif
+#ifdef PIPEX_L2TP
+       if (session->protocol == PIPEX_PROTO_L2TP)
+               LIST_REMOVE(session, peer_addr_chain);
+#endif
 
        /* if final session is destroyed, stop timer */
        if (LIST_EMPTY(&pipex_session_list))

Reply via email to