diff options
| author | Lutz Donnerhacke <donner@FreeBSD.org> | 2021-06-13 22:00:35 +0000 |
|---|---|---|
| committer | Lutz Donnerhacke <donner@FreeBSD.org> | 2021-06-16 21:55:35 +0000 |
| commit | ad2c95130b4baceca8f314e4f1d966fcac8b96b9 (patch) | |
| tree | 7fb426c963c2000aefe3e871d1cec3e5e5b64621 /sys/netgraph | |
| parent | 07ef7a03496563fddd51e891dbc9e7e5d0b1dccf (diff) | |
| download | src-ad2c95130b4baceca8f314e4f1d966fcac8b96b9.tar.gz src-ad2c95130b4baceca8f314e4f1d966fcac8b96b9.zip | |
netgraph/bridge: malloc without flags
During tests an assert was triggered and pointed to missing flags in
the newlink function of ng_bridge(4).
Reported by: markj
Reviewed by: markj
Differential Revision: https://reviews.freebsd.org/D30759
(cherry picked from commit 4c3280e58727e900d4c217054fe655e3512380f1)
Diffstat (limited to 'sys/netgraph')
| -rw-r--r-- | sys/netgraph/ng_bridge.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/sys/netgraph/ng_bridge.c b/sys/netgraph/ng_bridge.c index baedeb911e6d..3f3d8b3c406e 100644 --- a/sys/netgraph/ng_bridge.c +++ b/sys/netgraph/ng_bridge.c @@ -395,9 +395,7 @@ ng_bridge_newhook(node_p node, hook_p hook, const char *name) if(NG_PEER_NODE(hook) == node) return (ELOOP); - link = malloc(sizeof(*link), M_NETGRAPH_BRIDGE, M_ZERO); - if (link == NULL) - return (ENOMEM); + link = malloc(sizeof(*link), M_NETGRAPH_BRIDGE, M_WAITOK | M_ZERO); link->stats.recvOctets = counter_u64_alloc(M_WAITOK); link->stats.recvPackets = counter_u64_alloc(M_WAITOK); |