avoid NULL deref if malloc fails
authorjsg <jsg@openbsd.org>
Mon, 19 Aug 2024 08:22:30 +0000 (08:22 +0000)
committerjsg <jsg@openbsd.org>
Mon, 19 Aug 2024 08:22:30 +0000 (08:22 +0000)
ok stsp@

sys/dev/ic/qwx.c
sys/dev/ic/qwz.c

index 684210a..bd01745 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: qwx.c,v 1.65 2024/08/18 07:34:45 stsp Exp $   */
+/*     $OpenBSD: qwx.c,v 1.66 2024/08/19 08:22:30 jsg Exp $    */
 
 /*
  * Copyright 2023 Stefan Sperling <stsp@openbsd.org>
@@ -173,7 +173,8 @@ qwx_node_alloc(struct ieee80211com *ic)
        struct qwx_node *nq;
 
        nq = malloc(sizeof(struct qwx_node), M_DEVBUF, M_NOWAIT | M_ZERO);
-       nq->peer.peer_id = HAL_INVALID_PEERID;
+       if (nq != NULL)
+               nq->peer.peer_id = HAL_INVALID_PEERID;
        return (struct ieee80211_node *)nq;
 }
 
index 063631b..0de2e61 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: qwz.c,v 1.4 2024/08/16 00:26:54 patrick Exp $ */
+/*     $OpenBSD: qwz.c,v 1.5 2024/08/19 08:22:30 jsg Exp $     */
 
 /*
  * Copyright 2023 Stefan Sperling <stsp@openbsd.org>
@@ -172,7 +172,8 @@ qwz_node_alloc(struct ieee80211com *ic)
        struct qwz_node *nq;
 
        nq = malloc(sizeof(struct qwz_node), M_DEVBUF, M_NOWAIT | M_ZERO);
-       nq->peer.peer_id = HAL_INVALID_PEERID;
+       if (nq != NULL)
+               nq->peer.peer_id = HAL_INVALID_PEERID;
        return (struct ieee80211_node *)nq;
 }