Skip to content

Commit

Permalink
apparmor: remove duplicate macro list_entry_is_head()
Browse files Browse the repository at this point in the history
Strangely I hadn't had noticed the existence of the list_entry_is_head()
in apparmor code when added the same one in the list.h.  Luckily it's
fully identical and didn't break builds.  In any case we don't need a
duplicate anymore, thus remove it from apparmor code.

Link: https://lkml.kernel.org/r/[email protected]
Fixes: e130816 ("include/linux/list.h: add a macro to test if entry is pointing to the head")
Signed-off-by: Andy Shevchenko <[email protected]>
Acked-by: John Johansen <[email protected]>
Cc: James Morris <[email protected]>
Cc: "Serge E . Hallyn " <[email protected]>
Signed-off-by: Andrew Morton <[email protected]>
Signed-off-by: Linus Torvalds <[email protected]>
  • Loading branch information
andy-shev authored and torvalds committed Dec 16, 2020
1 parent 33fb626 commit 9801ca2
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions security/apparmor/apparmorfs.c
Original file line number Diff line number Diff line change
Expand Up @@ -2046,9 +2046,6 @@ int __aafs_ns_mkdir(struct aa_ns *ns, struct dentry *parent, const char *name,
return error;
}


#define list_entry_is_head(pos, head, member) (&pos->member == (head))

/**
* __next_ns - find the next namespace to list
* @root: root namespace to stop search at (NOT NULL)
Expand Down

0 comments on commit 9801ca2

Please sign in to comment.