setfiles: switch bb_info_msg to printf
authorDenys Vlasenko <vda.linux@googlemail.com>
Wed, 30 Mar 2016 16:17:35 +0000 (18:17 +0200)
committerDenys Vlasenko <vda.linux@googlemail.com>
Wed, 30 Mar 2016 16:17:35 +0000 (18:17 +0200)
Presumably, bb_info_msg was used here for syslog logging (-l),
but there is no actual code to activate syslog logging.
Added a TODO note on that, so that selinux users would notice
and fix if needed.

Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
selinux/setfiles.c

index c974c4a9d4e00aa344dbe0f9d61c423c98635cad..de99dfe442de487b83ada72072ec98debc67e80a 100644 (file)
@@ -17,6 +17,7 @@
 //usage:       )
 //usage:     "\n       -d      Show which specification matched each file"
 //usage:     "\n       -l      Log changes in file labels to syslog"
+//TODO: log to syslog is not yet implemented, it goes to stdout only now
 //usage:     "\n       -n      Don't change any file labels"
 //usage:     "\n       -q      Suppress warnings"
 //usage:     "\n       -r DIR  Use an alternate root path"
@@ -383,16 +384,16 @@ static int restore(const char *file)
                 * the user has changed but the role and type are the
                 * same.  For "-vv", emit everything. */
                if (verbose > 1 || !user_only_changed) {
-                       bb_info_msg("%s: reset %s context %s->%s",
+                       printf("%s: reset %s context %s->%s\n",
                                applet_name, my_file, context ? context : "", newcon);
                }
        }
 
        if (FLAG_l_take_log && !user_only_changed) {
                if (context)
-                       bb_info_msg("relabeling %s from %s to %s", my_file, context, newcon);
+                       printf("relabeling %s from %s to %s\n", my_file, context, newcon);
                else
-                       bb_info_msg("labeling %s to %s", my_file, newcon);
+                       printf("labeling %s to %s\n", my_file, newcon);
        }
 
        if (outfile && !user_only_changed)