Skip to content

Commit

Permalink
netfilter: ipt_ULOG: Move away from NLMSG_PUT().
Browse files Browse the repository at this point in the history
And use nlmsg_data() while we're here too.

Signed-off-by: David S. Miller <davem@davemloft.net>
  • Loading branch information
David S. Miller committed Jun 27, 2012
1 parent d106352 commit c2bd4ba
Showing 1 changed file with 8 additions and 7 deletions.
15 changes: 8 additions & 7 deletions net/ipv4/netfilter/ipt_ULOG.c
Original file line number Diff line number Diff line change
Expand Up @@ -196,12 +196,15 @@ static void ipt_ulog_packet(unsigned int hooknum,

pr_debug("qlen %d, qthreshold %Zu\n", ub->qlen, loginfo->qthreshold);

/* NLMSG_PUT contains a hidden goto nlmsg_failure !!! */
nlh = NLMSG_PUT(ub->skb, 0, ub->qlen, ULOG_NL_EVENT,
sizeof(*pm)+copy_len);
nlh = nlmsg_put(ub->skb, 0, ub->qlen, ULOG_NL_EVENT,
sizeof(*pm)+copy_len, 0);
if (!nlh) {
pr_debug("error during nlmsg_put\n");
goto out_unlock;
}
ub->qlen++;

pm = NLMSG_DATA(nlh);
pm = nlmsg_data(nlh);

/* We might not have a timestamp, get one */
if (skb->tstamp.tv64 == 0)
Expand Down Expand Up @@ -261,13 +264,11 @@ static void ipt_ulog_packet(unsigned int hooknum,
nlh->nlmsg_type = NLMSG_DONE;
ulog_send(groupnum);
}

out_unlock:
spin_unlock_bh(&ulog_lock);

return;

nlmsg_failure:
pr_debug("error during NLMSG_PUT\n");
alloc_failure:
pr_debug("Error building netlink message\n");
spin_unlock_bh(&ulog_lock);
Expand Down

0 comments on commit c2bd4ba

Please sign in to comment.