[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [RISU 4/9] Adjust tracefile open for write
From: |
Peter Maydell |
Subject: |
Re: [RISU 4/9] Adjust tracefile open for write |
Date: |
Mon, 18 May 2020 16:53:42 +0100 |
On Wed, 13 May 2020 at 19:10, Richard Henderson
<address@hidden> wrote:
>
> Truncate the new output file. Rely on umask to remove
> group+other file permissions, if desired.
>
> Signed-off-by: Richard Henderson <address@hidden>
> ---
> 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);
> }
I dunno why we were giving it execute permissions...
Reviewed-by: Peter Maydell <address@hidden>
thanks
-- PMM
- [RISU 0/9] risu cleanups and improvements, Richard Henderson, 2020/05/13
- [RISU 1/9] Use bool for tracing variables, Richard Henderson, 2020/05/13
- [RISU 2/9] Unify master_fd and apprentice_fd to comm_fd, Richard Henderson, 2020/05/13
- [RISU 3/9] Hoist trace file opening, Richard Henderson, 2020/05/13
- [RISU 6/9] Add magic and size to the trace header, Richard Henderson, 2020/05/13
- [RISU 4/9] Adjust tracefile open for write, Richard Henderson, 2020/05/13
- Re: [RISU 4/9] Adjust tracefile open for write,
Peter Maydell <=
- [RISU 8/9] aarch64: Reorg sve reginfo to save space, Richard Henderson, 2020/05/13
- [RISU 7/9] Compute reginfo_size based on the reginfo, Richard Henderson, 2020/05/13
- [RISU 5/9] Use EXIT_FAILURE, EXIT_SUCCESS, Richard Henderson, 2020/05/13
- [RISU 9/9] Add --dump option to inspect trace files, Richard Henderson, 2020/05/13
- Re: [RISU 0/9] risu cleanups and improvements, Peter Maydell, 2020/05/18