Page MenuHomeFreeBSD

elfcopy: stop leaving temporary files on error
ClosedPublic

Authored by crees on Feb 13 2021, 4:33 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Apr 6, 6:18 AM
Unknown Object (File)
Mar 9 2024, 10:54 AM
Unknown Object (File)
Mar 9 2024, 10:50 AM
Unknown Object (File)
Mar 9 2024, 5:08 AM
Unknown Object (File)
Mar 7 2024, 8:41 PM
Unknown Object (File)
Mar 7 2024, 8:32 PM
Unknown Object (File)
Jan 14 2024, 3:44 AM
Unknown Object (File)
Jan 3 2024, 12:34 PM
Subscribers

Details

Summary

Temporary files were not cleaned up, resulting in $TMPDIR or even
the current directory becoming littered with ecp.* files.

This happened with error and even sometimes on success!

Test Plan

make buildworld

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

crees requested review of this revision.Feb 13 2021, 4:33 PM

Hold for a bit- it errs on the verbose side, so in the test suite we get unnecessary stderr output. Will add extra checks.

elftoolchain: stop leaving tempfiles on error

Temporary files were not cleaned up, resulting in $TMPDIR or even
the current directory becoming littered with ecp.* files.

This happened with error and even sometimes on success!

Obtained from: https://sourceforge.net/p/elftoolchain/code/3918/

I think this should be ready to go- upstream has accepted it. Are you happy with it?

Ah sorry, totally forgotten about this. It seems to be fine, let's get it in then.

This revision is now accepted and ready to land.Mar 3 2021, 4:42 PM
This revision was automatically updated to reflect the committed changes.

Is there a reason why this has not been back ported to stable/12? Here are two issues suffering from the lack: