Page MenuHomeFreeBSD

D42330.diff
No OneTemporary

D42330.diff

diff --git a/lib/libpfctl/libpfctl.c b/lib/libpfctl/libpfctl.c
--- a/lib/libpfctl/libpfctl.c
+++ b/lib/libpfctl/libpfctl.c
@@ -196,6 +196,8 @@
start ? PFNL_CMD_START : PFNL_CMD_STOP);
hdr = snl_finalize_msg(&nw);
+ if (hdr == NULL)
+ return (ENOMEM);
seq_id = hdr->nlmsg_seq;
snl_send_message(&ss, hdr);
@@ -730,6 +732,8 @@
strlcpy(ri->name, nvlist_get_string(nvl, "name"),
PF_ANCHOR_NAME_SIZE);
+ nvlist_destroy(nvl);
+
return (0);
}
@@ -828,8 +832,8 @@
pfctl_nv_add_rule_addr(nvl, "ipdst", &r->ipdst);
labelcount = 0;
- while (r->label[labelcount][0] != 0 &&
- labelcount < PF_RULE_MAX_LABEL_COUNT) {
+ while (labelcount < PF_RULE_MAX_LABEL_COUNT &&
+ r->label[labelcount][0] != 0) {
nvlist_append_string_array(nvl, "labels",
r->label[labelcount]);
labelcount++;
@@ -1208,6 +1212,8 @@
hdr = snl_create_genl_msg_request(&nw, family_id, PFNL_CMD_GETCREATORS);
hdr->nlmsg_flags |= NLM_F_DUMP;
hdr = snl_finalize_msg(&nw);
+ if (hdr == NULL)
+ return (ENOMEM);
uint32_t seq_id = hdr->nlmsg_seq;
snl_send_message(ss, hdr);
@@ -1362,6 +1368,8 @@
snl_add_msg_attr_ip6(&nw, PF_ST_FILTER_MASK, &filter->mask.v6);
hdr = snl_finalize_msg(&nw);
+ if (hdr == NULL)
+ return (ENOMEM);
uint32_t seq_id = hdr->nlmsg_seq;
@@ -1417,7 +1425,7 @@
memcpy(new, s, sizeof(*s));
- TAILQ_INSERT_TAIL(&states->states, s, entry);
+ TAILQ_INSERT_TAIL(&states->states, new, entry);
return (0);
}

File Metadata

Mime Type
text/plain
Expires
Sat, Feb 22, 4:30 PM (2 h, 38 m)
Storage Engine
blob
Storage Format
Raw Data
Storage Handle
16774918
Default Alt Text
D42330.diff (1 KB)

Event Timeline