patchelf breaks hardlinks about patchelf

[darealshinji]

patchelf writes the changes in a new temporary file and replaces the original with the new one, that's why this happens.
https://github.com/NixOS/patchelf/blob/master/src/patchelf.cc#L378

[sjackman]

Breaking hardlinks is rather unexpected behaviour. Can writeFile be changed to write directly to the file, rather than through a temp file?

[darealshinji]

I found the solution! This is what "static void writeFile()" should look like:

static void writeFile(string fileName, mode_t fileMode)
{
    int fd = open(fileName.c_str(), O_TRUNC | O_WRONLY, 0700);
    if (fd == -1)
        error("open");

    if (write(fd, contents, fileSize) != fileSize)
        error("write");

    if (close(fd) != 0)
        error("close");
}

darealshinji@bed9cea

[sjackman]

Excellent! Thanks. I'll test it soon. Without O_CREAT you can remove , 0700 as it has no effect.

[sjackman]

@edolstra Can you please merge this patch from @darealshinji to fix this issue? Thanks!
darealshinji@bed9cea

[sjackman]

Great. Thanks. Will there be a new stable release that includes this fix?