efibootdump: fopen: use 'e' to set O_CLOEXEC

[matdibu]

src/efibootdump.c:230:23: warning: use 'fopen' mode 'e' to set O_CLOEXEC [android-cloexec-fopen]
                f = fopen(filename, "r");
                                    ^~~
                                    "re"

[frozencemetery]

efibootmgr doesn't exec, so the compiler is just being difficult... what compiler is it?

[matdibu]

this makes it so it automatically closes file descriptors on fork/exec It's good practice to avoid leaked file descriptors while not exactly happening here, it doesn't hurt to add it

[frozencemetery]

That's true, but now I'm concerned because you're not telling me what compiler you're encountering this with.

If you've not read this post on how to write effective commit messages, I recommend it. The important part for this conversation (and your other PRs) is to establish why a change is happening so that we have that information for the future.

[frozencemetery]

It's been several months; closing as inactive.