upstream: use error()+_exit() instead of fatal() to avoid running

cleanup handlers in child process; spotted via weird regress failures in
portable

OpenBSD-Commit-ID: 6902a9bb3987c7d347774444f7979b8a9ba7f412
This commit is contained in:
djm@openbsd.org 2019-11-27 05:00:17 +00:00 committed by Damien Miller
parent 70ec5e5e26
commit f79364baca

View File

@ -1,4 +1,4 @@
/* $OpenBSD: readpass.c,v 1.57 2019/11/15 00:32:40 djm Exp $ */
/* $OpenBSD: readpass.c,v 1.58 2019/11/27 05:00:17 djm Exp $ */
/*
* Copyright (c) 2001 Markus Friedl. All rights reserved.
*
@ -251,7 +251,8 @@ notify_start(int force_askpass, const char *fmt, ...)
closefrom(STDERR_FILENO + 1);
setenv("SSH_ASKPASS_PROMPT", "none", 1); /* hint to UI */
execlp(askpass, askpass, prompt, (char *)NULL);
fatal("%s: exec(%s): %s", __func__, askpass, strerror(errno));
error("%s: exec(%s): %s", __func__, askpass, strerror(errno));
_exit(1);
/* NOTREACHED */
}
if ((ret = calloc(1, sizeof(*ret))) == NULL) {