Check for execv(3) errors when reincarnating

When -reincarnate-reexec is given we run execv to restart FS.  If
argv[0] isn't a full pathname then execv is going to fail.  While not
common for a FS system started by init, this is a common occurrence
when FS is started from the shell.

Now if execv fails, we'll try execvp.  If that fails too then we'll
fall back on the normal reincarnation behavior.

Previously what would happen in that case is god would descend from
the heavens and become mortal.  Leaving heaven absent, all hope for
reincarnation was lost.

(That is, we'd simply return from reincarnate_protect and the
supervisor process would become the new instance of FS, so the trick
would only work once.)
This commit is contained in:
Travis Cross 2014-06-10 23:36:56 +00:00
parent 03202e3017
commit 12da85ef61
1 changed files with 13 additions and 1 deletions

View File

@ -399,7 +399,19 @@ static void reincarnate_protect(char **argv) {
sigaction(SIGTERM, &sa15_prev, NULL);
sigaction(SIGCHLD, &sa17_prev, NULL);
if (argv) {
execv(argv[0], argv); return;
if (execv(argv[0], argv) == -1) {
char buf[256];
fprintf(stderr, "Reincarnate execv() failed: %d %s\n", errno,
strerror_r(errno, buf, sizeof(buf)));
}
fprintf(stderr, "Trying reincarnate-reexec plan B...\n");
if (execvp(argv[0], argv) == -1) {
char buf[256];
fprintf(stderr, "Reincarnate execvp() failed: %d %s\n", errno,
strerror_r(errno, buf, sizeof(buf)));
}
fprintf(stderr, "Falling back to normal reincarnate behavior...\n");
goto refork;
} else goto refork;
}
goto rewait;