[CLEANUP] accept: replace some inappropriate Alert() calls with send_log()
Some Alert() messages were remaining in the accept() path, which they
would have no chance to be detected. Remove some of them (the impossible
ones) and replace the relevant ones with send_log() so that the admin
has a chance to catch them.
diff --git a/src/session.c b/src/session.c
index 5432266..d718695 100644
--- a/src/session.c
+++ b/src/session.c
@@ -57,10 +57,8 @@
struct http_txn *txn;
struct task *t;
- if (unlikely((s = pool_alloc2(pool2_session)) == NULL)) {
- Alert("out of memory in event_accept().\n");
+ if (unlikely((s = pool_alloc2(pool2_session)) == NULL))
goto out_close;
- }
/* minimum session initialization required for monitor mode below */
s->flags = 0;
@@ -89,10 +87,8 @@
LIST_ADDQ(&sessions, &s->list);
LIST_INIT(&s->back_refs);
- if (unlikely((t = task_new()) == NULL)) { /* disable this proxy for a while */
- Alert("out of memory in event_accept().\n");
+ if (unlikely((t = task_new()) == NULL))
goto out_free_session;
- }
s->term_trace = 0;
s->cli_addr = *addr;
@@ -211,10 +207,8 @@
s->store_count = 0;
/* Adjust some socket options */
- if (unlikely(fcntl(cfd, F_SETFL, O_NONBLOCK) == -1)) {
- Alert("accept(): cannot set the socket in non blocking mode. Giving up\n");
+ if (unlikely(fcntl(cfd, F_SETFL, O_NONBLOCK) == -1))
goto out_free_task;
- }
txn = &s->txn;
/* Those variables will be checked and freed if non-NULL in