Author: tuexen
Date: Wed Apr 22 12:47:46 2020
New Revision: 360193
URL: https://svnweb.freebsd.org/changeset/base/360193

Log:
  Improve input validation when processing AUTH chunks.
  
  Thanks to Natalie Silvanovich from Google for finding and reporting the
  issue found by her in the SCTP userland stack.
  
  MFC after:            3 days

Modified:
  head/sys/netinet/sctp_input.c

Modified: head/sys/netinet/sctp_input.c
==============================================================================
--- head/sys/netinet/sctp_input.c       Wed Apr 22 09:53:41 2020        
(r360192)
+++ head/sys/netinet/sctp_input.c       Wed Apr 22 12:47:46 2020        
(r360193)
@@ -2273,8 +2273,11 @@ sctp_process_cookie_new(struct mbuf *m, int iphlen, in
        if (auth_skipped) {
                struct sctp_auth_chunk *auth;
 
-               auth = (struct sctp_auth_chunk *)
-                   sctp_m_getptr(m, auth_offset, auth_len, auth_chunk_buf);
+               if (auth_len <= SCTP_PARAM_BUFFER_SIZE) {
+                       auth = (struct sctp_auth_chunk *)sctp_m_getptr(m, 
auth_offset, auth_len, auth_chunk_buf);
+               } else {
+                       auth = NULL;
+               }
                if ((auth == NULL) || sctp_handle_auth(stcb, auth, m, 
auth_offset)) {
                        /* auth HMAC failed, dump the assoc and packet */
                        SCTPDBG(SCTP_DEBUG_AUTH1,
_______________________________________________
svn-src-head@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"

Reply via email to