Fix Cycles animation denoising not working for files on a network drive.

Can't do cross-device renaming, so put temporary file in same folder as
output file.

1 files changed, 7 insertions, 5 deletions

diff --git a/intern/cycles/render/denoising.cpp b/intern/cycles/render/denoising.cpp
index 2fceee09c78..a89cb97c4d8 100644
--- a/intern/cycles/render/denoising.cpp
+++ b/intern/cycles/render/denoising.cpp
@@ -755,7 +755,9 @@ bool DenoiseImage::save_output(const string& out_filepath, string& error)
 
 	/* Write to temporary file path, so we denoise images in place and don't
 	 * risk destroying files when something goes wrong in file saving. */
-	string tmp_filepath = OIIO::Filesystem::temp_directory_path() + "/" + OIIO::Filesystem::unique_path() + ".exr";
+	string extension = OIIO::Filesystem::extension(out_filepath);
+	string unique_name = ".denoise-tmp-" + OIIO::Filesystem::unique_path();
+	string tmp_filepath = out_filepath + unique_name + extension;
 	unique_ptr<ImageOutput> out(ImageOutput::create(tmp_filepath));
 
 	if(!out) {
@@ -783,17 +785,17 @@ bool DenoiseImage::save_output(const string& out_filepath, string& error)
 	out.reset();
 
 	/* Copy temporary file to outputput filepath. */
-	if(ok && !OIIO::Filesystem::rename(tmp_filepath, out_filepath)) {
-		error = "Failed to save to file " + out_filepath + ": " + out->geterror();
+	string rename_error;
+	if(ok && !OIIO::Filesystem::rename(tmp_filepath, out_filepath, rename_error)) {
+		error = "Failed to move denoised image to " + out_filepath + ": " + rename_error;
 		ok = false;
 	}
 
 	if(!ok) {
 		OIIO::Filesystem::remove(tmp_filepath);
-		return false;
 	}
 
-	return true;
+	return ok;
 }
 
 /* File pattern handling and outer loop over frames */