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.

Sign up for GitHub

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

Jump to bottom

sql: updating the way to work with temporary files #5625

Open
LeonidVas opened this issue Dec 14, 2020 · 1 comment
Open

sql: updating the way to work with temporary files #5625

LeonidVas opened this issue Dec 14, 2020 · 1 comment
Labels
refactoring Code refactoring sql
Milestone
wishlist

Comments

@LeonidVas
Copy link
Contributor

The SQL module now uses some name generator to create temporary files. If this generator has weaknesses, it can have dire consequences(#5537).
The true way to work with temporary files in Linux is to use O_TMPFILE or tmpfile().
@LeonidVas LeonidVas added sql refactoring Code refactoring labels Dec 14, 2020
@LeonidVas
Copy link
Contributor Author

LeonidVas commented Dec 14, 2020 •
edited

I have no idea why this task needs to be done after fixing #5537 .
But @Korablev77 and @sergos think this might be helpful.

@kyukhin kyukhin added this to the wishlist milestone Jul 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
refactoring Code refactoring sql
Projects
None yet
Milestone
wishlist
Development

No branches or pull requests
2 participants
@kyukhin @LeonidVas