diff mbox series

[RISU,v2,04/17] Adjust tracefile open for write

Message ID 20200519025355.4420-5-richard.henderson@linaro.org
State Superseded
Headers show
Series risu cleanups and improvements | expand

Commit Message

Richard Henderson May 19, 2020, 2:53 a.m. UTC 
Truncate the new output file.  Rely on umask to remove
group+other file permissions, if desired.

Reviewed-by: Peter Maydell <peter.maydell@linaro.org>

Signed-off-by: Richard Henderson <richard.henderson@linaro.org>

---
 risu.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

-- 
2.20.1

Comments

Alex Bennée May 19, 2020, 6:24 p.m. UTC | #1 
Richard Henderson <richard.henderson@linaro.org> writes:

> Truncate the new output file.  Rely on umask to remove

> group+other file permissions, if desired.

>

> Reviewed-by: Peter Maydell <peter.maydell@linaro.org>

> Signed-off-by: Richard Henderson <richard.henderson@linaro.org>


Reviewed-by: Alex Bennée <alex.bennee@linaro.org>


> ---

>  risu.c | 2 +-

>  1 file changed, 1 insertion(+), 1 deletion(-)

>

> diff --git a/risu.c b/risu.c

> index 1c66885..f404d8f 100644

> --- a/risu.c

> +++ b/risu.c

> @@ -368,7 +368,7 @@ int main(int argc, char **argv)

>              comm_fd = ismaster ? STDOUT_FILENO : STDIN_FILENO;

>          } else {

>              if (ismaster) {

> -                comm_fd = open(trace_fn, O_WRONLY | O_CREAT, S_IRWXU);

> +                comm_fd = open(trace_fn, O_WRONLY | O_CREAT | O_TRUNC, 0666);

>              } else {

>                  comm_fd = open(trace_fn, O_RDONLY);

>              }



-- 
Alex Bennée
diff mbox series

Patch

diff --git a/risu.c b/risu.c
index 1c66885..f404d8f 100644
--- a/risu.c
+++ b/risu.c
@@ -368,7 +368,7 @@  int main(int argc, char **argv)
             comm_fd = ismaster ? STDOUT_FILENO : STDIN_FILENO;
         } else {
             if (ismaster) {
-                comm_fd = open(trace_fn, O_WRONLY | O_CREAT, S_IRWXU);
+                comm_fd = open(trace_fn, O_WRONLY | O_CREAT | O_TRUNC, 0666);
             } else {
                 comm_fd = open(trace_fn, O_RDONLY);
             }