Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

The entire Steam Client should honour TMPDIR, if set #11671

Open
Athanasius opened this issue Jan 19, 2025 · 0 comments
Open

The entire Steam Client should honour TMPDIR, if set #11671

Athanasius opened this issue Jan 19, 2025 · 0 comments

Comments

@Athanasius
Copy link

I had a steam client crash yesterday which caused a dump to be written. I had TMPDIR set to an alternate (and only accessible by my user) location, but this was ignored in favour of just writing in /tmp/dumps.

That caused my root partition to fill up, which I only noticed when a game failed to launch today. Took me a while to spot the issue, given I was looking all over / for where I could free up space. If my TMPDIR had been respected then the files would have been written to a different partition and not caused this issue.

That /tmp shouldn't be hard-coded, just a fallback if TMPDIR is not set.

This should apply to any need for a temporary file, be that in the core of the steam client, or in a helper script. Proper use of mktemp(1) or similar in any scripts would take care of this. For core code using the C API then mkdtemp(3) and mkstemp(3) (or a variant) should be utilised.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants