CLEANUP: checks: don't report report the fork() error twice
authorWilly Tarreau <w@1wt.eu>
Wed, 4 Oct 2017 13:19:26 +0000 (15:19 +0200)
committerWilly Tarreau <w@1wt.eu>
Wed, 4 Oct 2017 13:19:26 +0000 (15:19 +0200)
commitd7c3fbd5c304b695a73fc530b0472a276b1b4217
treec22ab31b53894c448bd236fd5dca31fd0bd2e435
parent1e62e2a780a9118e478ba05c4bdb546f133d7e4d
CLEANUP: checks: don't report report the fork() error twice

Upon fork() error, a first report is immediately made by connect_proc_chk()
via set_server_check_status(), then process_chk_proc() detects the error
code and makes up a dummy connection error to call chk_report_conn_err(),
which tries to retrieve the errno code from the connection, fails, then
saves the status message from the check, fails all "if" tests on its path
related to the connection then resets the check's state to the current one
with the current status message. All this useless chain is the only reason
why process checks require a connection! Let's simply get rid of this second
useless call.
src/checks.c