RepoMirrors/haproxy
1
0
Fork 0
mirror of http://git.haproxy.org/git/haproxy.git/ synced 2024-12-14 15:34:35 +00:00
Code Issues Packages Projects Releases Wiki Activity

MEDIUM: snapshot: merge the captured data after the descriptor

Browse Source 
Instead of having a separate area for the captured data, we now have a
contigous block made of the descriptor and the data. At the moment, since
the area is dynamically allocated, we can adjust its size to what is
needed, but the idea is to quickly switch to a pool and an LRU list.
This commit is contained in:
Willy Tarreau 2018-09-07 20:07:17 +02:00
parent c55015ee5b
commit 4bc7d90d3b
2 changed files with 10 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
include/types/proxy.h
View File

@ -225,7 +225,7 @@ struct error_snapshot {
/**** common part ****/
struct timeval when; /* date of this event, (tv_sec == 0) means "never" */
/* @16 */
char *buf; /* copy of the beginning of the message (may be NULL) */
void (*show)(struct buffer *, const struct error_snapshot *); /* dump function */
unsigned long long buf_ofs; /* relative position of the buffer's input inside its container */
/* @32 */
unsigned int buf_out; /* pending output bytes _before_ the buffer's input (0..buf->data-1) */
@ -239,10 +239,10 @@ struct error_snapshot {
unsigned int ev_id; /* event number (counter incremented for each capture) */
/* @68: 4 bytes hole here */
struct sockaddr_storage src; /* client's address */
void (*show)(struct buffer *, const struct error_snapshot *); /* dump function */
/**** protocol-specific part ****/
union error_snapshot_ctx ctx;
char buf[0]; /* copy of the beginning of the message for bufsize bytes */
};
struct email_alert {

19
src/proxy.c
View File

@ -1360,25 +1360,22 @@ void proxy_capture_error(struct proxy *proxy, int is_back,
ev_id = HA_ATOMIC_XADD(&error_snapshot_id, 1);
es = malloc(sizeof(*es));
buf_len = b_data(buf) - buf_out;
es = malloc(sizeof(*es) + buf_len);
if (!es)
return;
es->ev_id = ev_id;
es->buf_len = buf_len;
es->ev_id = ev_id;
buf_len = b_data(buf) - buf_out;
len1 = b_size(buf) - buf_len;
if (len1 > buf_len)
len1 = buf_len;
len2 = buf_len - len1;
es->buf_len = buf_len;
if (!es->buf)
es->buf = malloc(global.tune.bufsize);
if (es->buf) {
if (len1) {
memcpy(es->buf, b_peek(buf, buf_out), len1);
len2 = buf_len - len1;
if (len2)
memcpy(es->buf + len1, b_orig(buf), len2);
}
@ -2119,7 +2116,7 @@ static int cli_io_handler_show_errors(struct appctx *appctx)
}
/* OK, ptr >= 0, so we have to dump the current line */
while (es->buf && appctx->ctx.errors.ptr < es->buf_len && appctx->ctx.errors.ptr < global.tune.bufsize) {
while (appctx->ctx.errors.ptr < es->buf_len && appctx->ctx.errors.ptr < global.tune.bufsize) {
int newptr;
int newline;