Merge pull request #6785 from patpatbear/issue_#6696_fix_aof_child_whitelist_SIGUSR1

fix impl of aof-child whitelist SIGUSR1 feature.
This commit is contained in:
Salvatore Sanfilippo 2020-02-27 17:39:33 +01:00 committed by GitHub
commit 652c532292
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -1798,14 +1798,15 @@ void backgroundRewriteDoneHandler(int exitcode, int bysignal) {
serverLog(LL_VERBOSE,
"Background AOF rewrite signal handler took %lldus", ustime()-now);
} else if (!bysignal && exitcode != 0) {
server.aof_lastbgrewrite_status = C_ERR;
serverLog(LL_WARNING,
"Background AOF rewrite terminated with error");
} else {
/* SIGUSR1 is whitelisted, so we have a way to kill a child without
* tirggering an error condition. */
if (bysignal != SIGUSR1)
server.aof_lastbgrewrite_status = C_ERR;
serverLog(LL_WARNING,
"Background AOF rewrite terminated with error");
} else {
server.aof_lastbgrewrite_status = C_ERR;
serverLog(LL_WARNING,
"Background AOF rewrite terminated by signal %d", bysignal);