On 11/10/21 4:22 AM, Xi Ruoyao wrote:
Isn't this warning actually a glibc bug
<https://sourceware.org/bugzilla/show_bug.cgi?id=27476>?
However we can't assume the libc we are using is Glibc. Even if the
libc supports getcwd() with NULL argument, we are still leaking memory.
You could also free the memory by calling exit(2). Something is pretty
wrong if fixincludes cannot access a file it was told to process. So,
technically, you're right. Practically, no difference. But it's fine by
me to make the change. It does avoid a bug in a certain version of a
certain library.