From 18490162f74d0dca0106ee0996f47d5ed333cded Mon Sep 17 00:00:00 2001 From: Marek Roszko Date: Wed, 19 Aug 2020 23:46:25 -0400 Subject: [PATCH] Use a local tmp file for plotting Fix #5238 (cherry picked from commit de7a1b647c8bb21a6e3292d39920d73c7a96ca2e) --- common/plotters/GERBER_plotter.cpp | 5 ++--- common/plotters/PDF_plotter.cpp | 2 +- 2 files changed, 3 insertions(+), 4 deletions(-) diff --git a/common/plotters/GERBER_plotter.cpp b/common/plotters/GERBER_plotter.cpp index 8b19244d44..4046ca29c2 100644 --- a/common/plotters/GERBER_plotter.cpp +++ b/common/plotters/GERBER_plotter.cpp @@ -170,9 +170,8 @@ bool GERBER_PLOTTER::StartPlot() finalFile = outputFile; // the actual gerber file will be created later - // Create a temporary filename to store gerber file - // note tmpfile() does not work under Vista and W7 in user mode - m_workFilename = filename + wxT(".tmp"); + // Create a temp file in system temp to avoid potential network share buffer issues for the final read and save + m_workFilename = wxFileName::CreateTempFileName( "" ); workFile = wxFopen( m_workFilename, wxT( "wt" )); outputFile = workFile; wxASSERT( outputFile ); diff --git a/common/plotters/PDF_plotter.cpp b/common/plotters/PDF_plotter.cpp index 6286bc83e6..a2b8b399e9 100644 --- a/common/plotters/PDF_plotter.cpp +++ b/common/plotters/PDF_plotter.cpp @@ -495,7 +495,7 @@ int PDF_PLOTTER::startPdfStream(int handle) "stream\n", handle + 1 ); // Open a temporary file to accumulate the stream - workFilename = filename + wxT(".tmp"); + workFilename = wxFileName::CreateTempFileName( "" ); workFile = wxFopen( workFilename, wxT( "w+b" )); wxASSERT( workFile ); return handle;