With only one caller of report_lsm_order(), insert the function's code
directly into the caller and ger rid of report_lsm_order().

Reviewed-by: Casey Schaufler <ca...@schaufler-ca.com>
Signed-off-by: Paul Moore <p...@paul-moore.com>
---
 security/lsm_init.c | 33 ++++++++++++---------------------
 1 file changed, 12 insertions(+), 21 deletions(-)

diff --git a/security/lsm_init.c b/security/lsm_init.c
index 18828a65c364..09afa7ad719e 100644
--- a/security/lsm_init.c
+++ b/security/lsm_init.c
@@ -291,26 +291,6 @@ static void __init ordered_lsm_parse(const char *order, 
const char *origin)
        kfree(sep);
 }
 
-static void __init report_lsm_order(void)
-{
-       struct lsm_info **lsm, *early;
-       int first = 0;
-
-       pr_info("initializing lsm=");
-
-       /* Report each enabled LSM name, comma separated. */
-       lsm_early_for_each_raw(early) {
-               if (is_enabled(early))
-                       pr_cont("%s%s", first++ == 0 ? "" : ",", early->name);
-       }
-       lsm_order_for_each(lsm) {
-               if (is_enabled(*lsm))
-                       pr_cont("%s%s", first++ == 0 ? "" : ",", (*lsm)->name);
-       }
-
-       pr_cont("\n");
-}
-
 /**
  * lsm_early_cred - during initialization allocate a composite cred blob
  * @cred: the cred that needs a blob
@@ -341,7 +321,9 @@ static void __init lsm_early_task(struct task_struct *task)
 
 static void __init ordered_lsm_init(void)
 {
+       unsigned int first = 0;
        struct lsm_info **lsm;
+       struct lsm_info *early;
 
        if (chosen_lsm_order) {
                if (chosen_major_lsm) {
@@ -357,7 +339,16 @@ static void __init ordered_lsm_init(void)
                lsm_prepare(*lsm);
        }
 
-       report_lsm_order();
+       pr_info("initializing lsm=");
+       lsm_early_for_each_raw(early) {
+               if (is_enabled(early))
+                       pr_cont("%s%s", first++ == 0 ? "" : ",", early->name);
+       }
+       lsm_order_for_each(lsm) {
+               if (is_enabled(*lsm))
+                       pr_cont("%s%s", first++ == 0 ? "" : ",", (*lsm)->name);
+       }
+       pr_cont("\n");
 
        init_debug("cred blob size       = %d\n", blob_sizes.lbs_cred);
        init_debug("file blob size       = %d\n", blob_sizes.lbs_file);
-- 
2.50.1


Reply via email to