← Back to team overview

dolfin team mailing list archive

Re: fix for temp files

 

On Mon, Oct 20, 2008 at 07:43:50PM +0200, Joachim Berdal Haga wrote:
> The previous fix c2407fd700ac for cleaning up /tmp droppings was incomplete
> since it only applied the "patch" to one of two similar code chunks.
> 
> This one tries to do it properly by moving common stuff to a separate
> function. Also fixes the one Hake identified. And the case where clean-up
> fails because exit() is called deep inside plot() somewhere.
> 
> Also fixes a potential security issue (execution of unsafe temp file) and
> some windows stuff (fixes that were previously only done for one of the two
> chunks).
> 
> Please look it over before applying, as it has only been lightly tested.

I trust you know what you're doing... and it seems to work... :-)

> Unsafe temp files are also used in other places, but this is probably the
> only one that is explicitly executed? Might still be worth fixing this issue
> everywhere, just to be sure.

Where? Do you mean the ODE solvers?

-- 
Anders

Attachment: signature.asc
Description: Digital signature


Follow ups