For completeness, I went through other uses of `fwrite()` and `g_file_set_contents()`: - `fwrite()` is used in tag manager to write some auxiliary file when generating tag files - pretty much OK since this is just a temporary file (TM doesn't have access to the rest of Geany ATM so it cannot use `utils_write_file()`) - `fwrite()` is used to write to the socket which has to be done this way - `g_file_set_contents()` is used to write the temporary `geany_run_script_XXXXXX.sh` script - could probably be converted to use `utils_write_file()` but right now it needs the `GError` for error messages which `utils_write_file()` does not return. Probably not worth the work since it's just a temporary file (and the possible wrong permissions will only be within the `/tmp` directory and won't prevent creation of other `geany_run_script_XXXXXX.sh` since the suffix is random)
So no extra work needed IMO. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3950#issuecomment-2353065824 You are receiving this because you are subscribed to this thread. Message ID: <geany/geany/pull/3950/[email protected]>
