quic/qlog: create log files with O_EXCL

Avoid confusing log corruption if two loggers try to write to the same file simultaneously. Change-Id: I3bfbcf56aa55c778ada0178d7c662c414878c9d1 Reviewed-on: https://go-review.googlesource.com/c/net/+/545577 Reviewed-by: Jonathan Amsterdam <[email protected]> LUCI-TryBot-Result: Go LUCI <[email protected]>
golang · Dec 18, 2023 · 2b416c3 · 2b416c3
1 parent 1e59a7e
commit 2b416c3
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/internal/quic/qlog/qlog.go b/internal/quic/qlog/qlog.go
@@ -180,7 +180,7 @@ func newTraceWriter(opts HandlerOptions, info TraceInfo) (io.WriteCloser, error)
 		if !filepath.IsLocal(filename) {
 			return nil, errors.New("invalid trace filename")
 		}
-		w, err = os.Create(filepath.Join(opts.Dir, filename))
+		w, err = os.OpenFile(filepath.Join(opts.Dir, filename), os.O_CREATE|os.O_EXCL|os.O_WRONLY, 0666)
 	} else {
 		err = errors.New("no log destination")
 	}