Commit 94191213 authored by Hong zhi guo's avatar Hong zhi guo Committed by David S. Miller

audit: replace obsolete NLMSG_* with type safe nlmsg_*

Signed-off-by: default avatarHong Zhiguo <honkiko@gmail.com>
Acked-by: default avatarThomas Graf <tgraf@suug.ch>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 573ce260
...@@ -58,7 +58,7 @@ ...@@ -58,7 +58,7 @@
#ifdef CONFIG_SECURITY #ifdef CONFIG_SECURITY
#include <linux/security.h> #include <linux/security.h>
#endif #endif
#include <linux/netlink.h> #include <net/netlink.h>
#include <linux/freezer.h> #include <linux/freezer.h>
#include <linux/tty.h> #include <linux/tty.h>
#include <linux/pid_namespace.h> #include <linux/pid_namespace.h>
...@@ -910,7 +910,7 @@ static void audit_receive_skb(struct sk_buff *skb) ...@@ -910,7 +910,7 @@ static void audit_receive_skb(struct sk_buff *skb)
{ {
struct nlmsghdr *nlh; struct nlmsghdr *nlh;
/* /*
* len MUST be signed for NLMSG_NEXT to be able to dec it below 0 * len MUST be signed for nlmsg_next to be able to dec it below 0
* if the nlmsg_len was not aligned * if the nlmsg_len was not aligned
*/ */
int len; int len;
...@@ -919,13 +919,13 @@ static void audit_receive_skb(struct sk_buff *skb) ...@@ -919,13 +919,13 @@ static void audit_receive_skb(struct sk_buff *skb)
nlh = nlmsg_hdr(skb); nlh = nlmsg_hdr(skb);
len = skb->len; len = skb->len;
while (NLMSG_OK(nlh, len)) { while (nlmsg_ok(nlh, len)) {
err = audit_receive_msg(skb, nlh); err = audit_receive_msg(skb, nlh);
/* if err or if this message says it wants a response */ /* if err or if this message says it wants a response */
if (err || (nlh->nlmsg_flags & NLM_F_ACK)) if (err || (nlh->nlmsg_flags & NLM_F_ACK))
netlink_ack(skb, nlh, err); netlink_ack(skb, nlh, err);
nlh = NLMSG_NEXT(nlh, len); nlh = nlmsg_next(nlh, len);
} }
} }
...@@ -1483,7 +1483,7 @@ void audit_log_end(struct audit_buffer *ab) ...@@ -1483,7 +1483,7 @@ void audit_log_end(struct audit_buffer *ab)
audit_log_lost("rate limit exceeded"); audit_log_lost("rate limit exceeded");
} else { } else {
struct nlmsghdr *nlh = nlmsg_hdr(ab->skb); struct nlmsghdr *nlh = nlmsg_hdr(ab->skb);
nlh->nlmsg_len = ab->skb->len - NLMSG_SPACE(0); nlh->nlmsg_len = ab->skb->len - NLMSG_HDRLEN;
if (audit_pid) { if (audit_pid) {
skb_queue_tail(&audit_skb_queue, ab->skb); skb_queue_tail(&audit_skb_queue, ab->skb);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment