Commit 14224039 authored by Richard Guy Briggs's avatar Richard Guy Briggs Committed by Paul Moore

audit: add gfp parameter to audit_log_nfcfg

Fixed an inconsistent use of GFP flags in nft_obj_notify() that used
GFP_KERNEL when a GFP flag was passed in to that function.  Given this
allocated memory was then used in audit_log_nfcfg() it led to an audit
of all other GFP allocations in net/netfilter/nf_tables_api.c and a
modification of audit_log_nfcfg() to accept a GFP parameter.
Reported-by: default avatarDan Carptenter <dan.carpenter@oracle.com>
Signed-off-by: default avatarRichard Guy Briggs <rgb@redhat.com>
Signed-off-by: default avatarPaul Moore <paul@paul-moore.com>
parent 8e6cf365
...@@ -404,7 +404,7 @@ extern void __audit_fanotify(unsigned int response); ...@@ -404,7 +404,7 @@ extern void __audit_fanotify(unsigned int response);
extern void __audit_tk_injoffset(struct timespec64 offset); extern void __audit_tk_injoffset(struct timespec64 offset);
extern void __audit_ntp_log(const struct audit_ntp_data *ad); extern void __audit_ntp_log(const struct audit_ntp_data *ad);
extern void __audit_log_nfcfg(const char *name, u8 af, unsigned int nentries, extern void __audit_log_nfcfg(const char *name, u8 af, unsigned int nentries,
enum audit_nfcfgop op); enum audit_nfcfgop op, gfp_t gfp);
static inline void audit_ipc_obj(struct kern_ipc_perm *ipcp) static inline void audit_ipc_obj(struct kern_ipc_perm *ipcp)
{ {
...@@ -542,10 +542,10 @@ static inline void audit_ntp_log(const struct audit_ntp_data *ad) ...@@ -542,10 +542,10 @@ static inline void audit_ntp_log(const struct audit_ntp_data *ad)
static inline void audit_log_nfcfg(const char *name, u8 af, static inline void audit_log_nfcfg(const char *name, u8 af,
unsigned int nentries, unsigned int nentries,
enum audit_nfcfgop op) enum audit_nfcfgop op, gfp_t gfp)
{ {
if (audit_enabled) if (audit_enabled)
__audit_log_nfcfg(name, af, nentries, op); __audit_log_nfcfg(name, af, nentries, op, gfp);
} }
extern int audit_n_rules; extern int audit_n_rules;
...@@ -683,7 +683,7 @@ static inline void audit_ptrace(struct task_struct *t) ...@@ -683,7 +683,7 @@ static inline void audit_ptrace(struct task_struct *t)
static inline void audit_log_nfcfg(const char *name, u8 af, static inline void audit_log_nfcfg(const char *name, u8 af,
unsigned int nentries, unsigned int nentries,
enum audit_nfcfgop op) enum audit_nfcfgop op, gfp_t gfp)
{ } { }
#define audit_n_rules 0 #define audit_n_rules 0
......
...@@ -2572,12 +2572,12 @@ void __audit_ntp_log(const struct audit_ntp_data *ad) ...@@ -2572,12 +2572,12 @@ void __audit_ntp_log(const struct audit_ntp_data *ad)
} }
void __audit_log_nfcfg(const char *name, u8 af, unsigned int nentries, void __audit_log_nfcfg(const char *name, u8 af, unsigned int nentries,
enum audit_nfcfgop op) enum audit_nfcfgop op, gfp_t gfp)
{ {
struct audit_buffer *ab; struct audit_buffer *ab;
char comm[sizeof(current->comm)]; char comm[sizeof(current->comm)];
ab = audit_log_start(audit_context(), GFP_KERNEL, AUDIT_NETFILTER_CFG); ab = audit_log_start(audit_context(), gfp, AUDIT_NETFILTER_CFG);
if (!ab) if (!ab)
return; return;
audit_log_format(ab, "table=%s family=%u entries=%u op=%s", audit_log_format(ab, "table=%s family=%u entries=%u op=%s",
......
...@@ -1047,7 +1047,7 @@ static int do_replace_finish(struct net *net, struct ebt_replace *repl, ...@@ -1047,7 +1047,7 @@ static int do_replace_finish(struct net *net, struct ebt_replace *repl,
vfree(counterstmp); vfree(counterstmp);
audit_log_nfcfg(repl->name, AF_BRIDGE, repl->nentries, audit_log_nfcfg(repl->name, AF_BRIDGE, repl->nentries,
AUDIT_XT_OP_REPLACE); AUDIT_XT_OP_REPLACE, GFP_KERNEL);
return ret; return ret;
free_unlock: free_unlock:
...@@ -1123,7 +1123,7 @@ static void __ebt_unregister_table(struct net *net, struct ebt_table *table) ...@@ -1123,7 +1123,7 @@ static void __ebt_unregister_table(struct net *net, struct ebt_table *table)
list_del(&table->list); list_del(&table->list);
mutex_unlock(&ebt_mutex); mutex_unlock(&ebt_mutex);
audit_log_nfcfg(table->name, AF_BRIDGE, table->private->nentries, audit_log_nfcfg(table->name, AF_BRIDGE, table->private->nentries,
AUDIT_XT_OP_UNREGISTER); AUDIT_XT_OP_UNREGISTER, GFP_KERNEL);
EBT_ENTRY_ITERATE(table->private->entries, table->private->entries_size, EBT_ENTRY_ITERATE(table->private->entries, table->private->entries_size,
ebt_cleanup_entry, net, NULL); ebt_cleanup_entry, net, NULL);
if (table->private->nentries) if (table->private->nentries)
...@@ -1218,7 +1218,7 @@ int ebt_register_table(struct net *net, const struct ebt_table *input_table, ...@@ -1218,7 +1218,7 @@ int ebt_register_table(struct net *net, const struct ebt_table *input_table,
} }
audit_log_nfcfg(repl->name, AF_BRIDGE, repl->nentries, audit_log_nfcfg(repl->name, AF_BRIDGE, repl->nentries,
AUDIT_XT_OP_REGISTER); AUDIT_XT_OP_REGISTER, GFP_KERNEL);
return ret; return ret;
free_unlock: free_unlock:
mutex_unlock(&ebt_mutex); mutex_unlock(&ebt_mutex);
......
...@@ -702,7 +702,8 @@ static void nf_tables_table_notify(const struct nft_ctx *ctx, int event) ...@@ -702,7 +702,8 @@ static void nf_tables_table_notify(const struct nft_ctx *ctx, int event)
ctx->table->use, ctx->table->use,
event == NFT_MSG_NEWTABLE ? event == NFT_MSG_NEWTABLE ?
AUDIT_NFT_OP_TABLE_REGISTER : AUDIT_NFT_OP_TABLE_REGISTER :
AUDIT_NFT_OP_TABLE_UNREGISTER); AUDIT_NFT_OP_TABLE_UNREGISTER,
GFP_KERNEL);
kfree(buf); kfree(buf);
if (!ctx->report && if (!ctx->report &&
...@@ -1448,7 +1449,8 @@ static void nf_tables_chain_notify(const struct nft_ctx *ctx, int event) ...@@ -1448,7 +1449,8 @@ static void nf_tables_chain_notify(const struct nft_ctx *ctx, int event)
ctx->chain->use, ctx->chain->use,
event == NFT_MSG_NEWCHAIN ? event == NFT_MSG_NEWCHAIN ?
AUDIT_NFT_OP_CHAIN_REGISTER : AUDIT_NFT_OP_CHAIN_REGISTER :
AUDIT_NFT_OP_CHAIN_UNREGISTER); AUDIT_NFT_OP_CHAIN_UNREGISTER,
GFP_KERNEL);
kfree(buf); kfree(buf);
if (!ctx->report && if (!ctx->report &&
...@@ -2724,7 +2726,8 @@ static void nf_tables_rule_notify(const struct nft_ctx *ctx, ...@@ -2724,7 +2726,8 @@ static void nf_tables_rule_notify(const struct nft_ctx *ctx,
rule->handle, rule->handle,
event == NFT_MSG_NEWRULE ? event == NFT_MSG_NEWRULE ?
AUDIT_NFT_OP_RULE_REGISTER : AUDIT_NFT_OP_RULE_REGISTER :
AUDIT_NFT_OP_RULE_UNREGISTER); AUDIT_NFT_OP_RULE_UNREGISTER,
GFP_KERNEL);
kfree(buf); kfree(buf);
if (!ctx->report && if (!ctx->report &&
...@@ -3737,7 +3740,8 @@ static void nf_tables_set_notify(const struct nft_ctx *ctx, ...@@ -3737,7 +3740,8 @@ static void nf_tables_set_notify(const struct nft_ctx *ctx,
set->field_count, set->field_count,
event == NFT_MSG_NEWSET ? event == NFT_MSG_NEWSET ?
AUDIT_NFT_OP_SET_REGISTER : AUDIT_NFT_OP_SET_REGISTER :
AUDIT_NFT_OP_SET_UNREGISTER); AUDIT_NFT_OP_SET_UNREGISTER,
gfp_flags);
kfree(buf); kfree(buf);
if (!ctx->report && if (!ctx->report &&
...@@ -4864,7 +4868,8 @@ static void nf_tables_setelem_notify(const struct nft_ctx *ctx, ...@@ -4864,7 +4868,8 @@ static void nf_tables_setelem_notify(const struct nft_ctx *ctx,
set->handle, set->handle,
event == NFT_MSG_NEWSETELEM ? event == NFT_MSG_NEWSETELEM ?
AUDIT_NFT_OP_SETELEM_REGISTER : AUDIT_NFT_OP_SETELEM_REGISTER :
AUDIT_NFT_OP_SETELEM_UNREGISTER); AUDIT_NFT_OP_SETELEM_UNREGISTER,
GFP_KERNEL);
kfree(buf); kfree(buf);
if (!ctx->report && !nfnetlink_has_listeners(net, NFNLGRP_NFTABLES)) if (!ctx->report && !nfnetlink_has_listeners(net, NFNLGRP_NFTABLES))
...@@ -5956,7 +5961,8 @@ static int nf_tables_dump_obj(struct sk_buff *skb, struct netlink_callback *cb) ...@@ -5956,7 +5961,8 @@ static int nf_tables_dump_obj(struct sk_buff *skb, struct netlink_callback *cb)
audit_log_nfcfg(buf, audit_log_nfcfg(buf,
family, family,
obj->handle, obj->handle,
AUDIT_NFT_OP_OBJ_RESET); AUDIT_NFT_OP_OBJ_RESET,
GFP_KERNEL);
kfree(buf); kfree(buf);
} }
...@@ -6071,13 +6077,14 @@ static int nf_tables_getobj(struct net *net, struct sock *nlsk, ...@@ -6071,13 +6077,14 @@ static int nf_tables_getobj(struct net *net, struct sock *nlsk,
reset = true; reset = true;
if (reset) { if (reset) {
char *buf = kasprintf(GFP_KERNEL, "%s:%llu;?:0", char *buf = kasprintf(GFP_ATOMIC, "%s:%llu;?:0",
table->name, table->handle); table->name, table->handle);
audit_log_nfcfg(buf, audit_log_nfcfg(buf,
family, family,
obj->handle, obj->handle,
AUDIT_NFT_OP_OBJ_RESET); AUDIT_NFT_OP_OBJ_RESET,
GFP_KERNEL);
kfree(buf); kfree(buf);
} }
...@@ -6156,7 +6163,7 @@ void nft_obj_notify(struct net *net, const struct nft_table *table, ...@@ -6156,7 +6163,7 @@ void nft_obj_notify(struct net *net, const struct nft_table *table,
{ {
struct sk_buff *skb; struct sk_buff *skb;
int err; int err;
char *buf = kasprintf(GFP_KERNEL, "%s:%llu;?:0", char *buf = kasprintf(gfp, "%s:%llu;?:0",
table->name, table->handle); table->name, table->handle);
audit_log_nfcfg(buf, audit_log_nfcfg(buf,
...@@ -6164,7 +6171,8 @@ void nft_obj_notify(struct net *net, const struct nft_table *table, ...@@ -6164,7 +6171,8 @@ void nft_obj_notify(struct net *net, const struct nft_table *table,
obj->handle, obj->handle,
event == NFT_MSG_NEWOBJ ? event == NFT_MSG_NEWOBJ ?
AUDIT_NFT_OP_OBJ_REGISTER : AUDIT_NFT_OP_OBJ_REGISTER :
AUDIT_NFT_OP_OBJ_UNREGISTER); AUDIT_NFT_OP_OBJ_UNREGISTER,
GFP_KERNEL);
kfree(buf); kfree(buf);
if (!report && if (!report &&
...@@ -6954,7 +6962,8 @@ static void nf_tables_flowtable_notify(struct nft_ctx *ctx, ...@@ -6954,7 +6962,8 @@ static void nf_tables_flowtable_notify(struct nft_ctx *ctx,
flowtable->hooknum, flowtable->hooknum,
event == NFT_MSG_NEWFLOWTABLE ? event == NFT_MSG_NEWFLOWTABLE ?
AUDIT_NFT_OP_FLOWTABLE_REGISTER : AUDIT_NFT_OP_FLOWTABLE_REGISTER :
AUDIT_NFT_OP_FLOWTABLE_UNREGISTER); AUDIT_NFT_OP_FLOWTABLE_UNREGISTER,
GFP_KERNEL);
kfree(buf); kfree(buf);
if (ctx->report && if (ctx->report &&
...@@ -7078,7 +7087,7 @@ static void nf_tables_gen_notify(struct net *net, struct sk_buff *skb, ...@@ -7078,7 +7087,7 @@ static void nf_tables_gen_notify(struct net *net, struct sk_buff *skb,
int err; int err;
audit_log_nfcfg("?:0;?:0", 0, net->nft.base_seq, audit_log_nfcfg("?:0;?:0", 0, net->nft.base_seq,
AUDIT_NFT_OP_GEN_REGISTER); AUDIT_NFT_OP_GEN_REGISTER, GFP_KERNEL);
if (nlmsg_report(nlh) && if (nlmsg_report(nlh) &&
!nfnetlink_has_listeners(net, NFNLGRP_NFTABLES)) !nfnetlink_has_listeners(net, NFNLGRP_NFTABLES))
......
...@@ -1410,7 +1410,8 @@ xt_replace_table(struct xt_table *table, ...@@ -1410,7 +1410,8 @@ xt_replace_table(struct xt_table *table,
audit_log_nfcfg(table->name, table->af, private->number, audit_log_nfcfg(table->name, table->af, private->number,
!private->number ? AUDIT_XT_OP_REGISTER : !private->number ? AUDIT_XT_OP_REGISTER :
AUDIT_XT_OP_REPLACE); AUDIT_XT_OP_REPLACE,
GFP_KERNEL);
return private; return private;
} }
EXPORT_SYMBOL_GPL(xt_replace_table); EXPORT_SYMBOL_GPL(xt_replace_table);
...@@ -1473,7 +1474,7 @@ void *xt_unregister_table(struct xt_table *table) ...@@ -1473,7 +1474,7 @@ void *xt_unregister_table(struct xt_table *table)
list_del(&table->list); list_del(&table->list);
mutex_unlock(&xt[table->af].mutex); mutex_unlock(&xt[table->af].mutex);
audit_log_nfcfg(table->name, table->af, private->number, audit_log_nfcfg(table->name, table->af, private->number,
AUDIT_XT_OP_UNREGISTER); AUDIT_XT_OP_UNREGISTER, GFP_KERNEL);
kfree(table); kfree(table);
return private; return private;
......
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