Commit 4a3eb726 authored by Richard Guy Briggs's avatar Richard Guy Briggs Committed by Eric Paris

audit: rename the misleading audit_get_context() to audit_take_context()

"get" usually implies incrementing a refcount into a structure to indicate a
reference being held by another part of code.

Change this function name to indicate it is in fact being taken from it,
returning the value while clearing it in the supplying structure.
Signed-off-by: default avatarRichard Guy Briggs <rgb@redhat.com>
parent 4b588411
...@@ -811,7 +811,8 @@ void audit_filter_inodes(struct task_struct *tsk, struct audit_context *ctx) ...@@ -811,7 +811,8 @@ void audit_filter_inodes(struct task_struct *tsk, struct audit_context *ctx)
rcu_read_unlock(); rcu_read_unlock();
} }
static inline struct audit_context *audit_get_context(struct task_struct *tsk, /* Transfer the audit context pointer to the caller, clearing it in the tsk's struct */
static inline struct audit_context *audit_take_context(struct task_struct *tsk,
int return_valid, int return_valid,
long return_code) long return_code)
{ {
...@@ -1474,7 +1475,7 @@ void __audit_free(struct task_struct *tsk) ...@@ -1474,7 +1475,7 @@ void __audit_free(struct task_struct *tsk)
{ {
struct audit_context *context; struct audit_context *context;
context = audit_get_context(tsk, 0, 0); context = audit_take_context(tsk, 0, 0);
if (!context) if (!context)
return; return;
...@@ -1568,7 +1569,7 @@ void __audit_syscall_exit(int success, long return_code) ...@@ -1568,7 +1569,7 @@ void __audit_syscall_exit(int success, long return_code)
else else
success = AUDITSC_FAILURE; success = AUDITSC_FAILURE;
context = audit_get_context(tsk, success, return_code); context = audit_take_context(tsk, success, return_code);
if (!context) if (!context)
return; return;
......
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