On Tue, 17 Jun 2014 16:05:09 -0400 Neil Horman <[email protected]> wrote:
> current, the tmon umask value is set to 0, which mean whatever the Currently, ..., which means ... > permission mask in the shell are when starting tmon in daemon mode > are what the permissions of any created files will be. We should > likely set something more explicit, so lets go with the usual 022 > Just some grammar suggestion. Otherwise, Acked-by: Jacob Pan <[email protected]> > Signed-off-by: Neil Horman <[email protected]> > CC: Zhang Rui <[email protected]> > CC: Jacob Pan <[email protected]> > --- > tools/thermal/tmon/tmon.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/tools/thermal/tmon/tmon.c b/tools/thermal/tmon/tmon.c > index 059e0be..09b7c32 100644 > --- a/tools/thermal/tmon/tmon.c > +++ b/tools/thermal/tmon/tmon.c > @@ -355,7 +355,7 @@ static void start_daemon_mode() > disable_tui(); > > /* change the file mode mask */ > - umask(0); > + umask(S_IWGRP | S_IWOTH); > > /* new SID for the daemon process */ > sid = setsid(); [Jacob Pan] -- Thanks, Jacob -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
