missing fsync in atomic create

[photoszzt]

I think this code is not atomic. https://github.com/StanfordSNR/gg/blob/febdf22643655bf9a89f7ddc8b5849dc7e8f4d9e/src/util/path.cc#L188-L209 Two fsync is missing. You need to fsync the temp file and fsync the containing directory after the rename. See: https://lwn.net/Articles/457667/

[JustinAzoff]

I don't thing consistency in the face of a kernel panic or power failure is a requirement based on how that code is used. Adding 2 fsyncs to every file creation would just slow builds down.

[photoszzt]

@JustinAzoff The problem is if I don't put these two fsyncs, the program fails. It complaints that the file is missing when the file is created but the directory is not fsync.