]> www.infradead.org Git - users/hch/misc.git/commitdiff
net/sctp: fix a null dereference in sctp_disposition sctp_sf_do_5_1D_ce()
authorAlexandr Sapozhnikov <alsp705@gmail.com>
Thu, 2 Oct 2025 09:14:47 +0000 (12:14 +0300)
committerJakub Kicinski <kuba@kernel.org>
Mon, 6 Oct 2025 18:07:20 +0000 (11:07 -0700)
If new_asoc->peer.adaptation_ind=0 and sctp_ulpevent_make_authkey=0
and sctp_ulpevent_make_authkey() returns 0, then the variable
ai_ev remains zero and the zero will be dereferenced
in the sctp_ulpevent_free() function.

Signed-off-by: Alexandr Sapozhnikov <alsp705@gmail.com>
Acked-by: Xin Long <lucien.xin@gmail.com>
Fixes: 30f6ebf65bc4 ("sctp: add SCTP_AUTH_NO_AUTH type for AUTHENTICATION_EVENT")
Link: https://patch.msgid.link/20251002091448.11-1-alsp705@gmail.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
net/sctp/sm_statefuns.c

index 4cb8f393434d0c0b106fbe6fc0f93e5b7cd69708..3755ba079d077c347b548c63998f60bcae786a49 100644 (file)
@@ -886,7 +886,8 @@ enum sctp_disposition sctp_sf_do_5_1D_ce(struct net *net,
        return SCTP_DISPOSITION_CONSUME;
 
 nomem_authev:
-       sctp_ulpevent_free(ai_ev);
+       if (ai_ev)
+               sctp_ulpevent_free(ai_ev);
 nomem_aiev:
        sctp_ulpevent_free(ev);
 nomem_ev: