Discussion:
[PATCH] audit: fixed a braces coding style issue
(too old to reply)
Paul Moore
2018-09-22 20:21:43 UTC
Permalink
On Fri, Sep 21, 2018 at 7:38 PM Przemysław Głębocki
Fixed a coding style issue.
---
kernel/audit_tree.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
I generally dislike accepting coding style patches from people who
haven't made any significant code contributions. Would you be
interested in participating in upstream audit development? We have a
number of open issues that you could take a look at if you are
interested ...

* https://github.com/linux-audit/audit-kernel/issues
diff --git a/kernel/audit_tree.c b/kernel/audit_tree.c
index ea43181cde4a..3d0e45fa765d 100644
--- a/kernel/audit_tree.c
+++ b/kernel/audit_tree.c
@@ -112,10 +112,9 @@ static void free_chunk(struct audit_chunk *chunk)
{
int i;
- for (i = 0; i < chunk->count; i++) {
+ for (i = 0; i < chunk->count; i++)
if (chunk->owners[i].owner)
put_tree(chunk->owners[i].owner);
- }
kfree(chunk);
}
--
2.17.1
--
paul moore
www.paul-moore.com
Przemysław Głębocki
2018-09-22 12:38:31 UTC
Permalink
Fixed a coding style issue.

Signed-off-by: Przemysław Głębocki <***@gmail.com>
---
kernel/audit_tree.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/kernel/audit_tree.c b/kernel/audit_tree.c
index ea43181cde4a..3d0e45fa765d 100644
--- a/kernel/audit_tree.c
+++ b/kernel/audit_tree.c
@@ -112,10 +112,9 @@ static void free_chunk(struct audit_chunk *chunk)
{
int i;

- for (i = 0; i < chunk->count; i++) {
+ for (i = 0; i < chunk->count; i++)
if (chunk->owners[i].owner)
put_tree(chunk->owners[i].owner);
- }
kfree(chunk);
}
--
2.17.1
Loading...