mirror of
https://git.haproxy.org/git/haproxy.git/
synced 2026-05-04 20:46:11 +02:00
BUG/MEDIUM: http: the "http_first_req" sample fetch could crash without a steeam
Since commit 5f940703b3 ("MINOR: log: Don't depends on a stream to process
samples in log-format string") it has become quite obvious that a few sample
fetch functions and converters were still heavily dependent on the presence
of a stream without testing for it.
The http_first_req sample fetch function, if called without a stream, will
result in a crash.
This fix adds a check for the stream's existence, and should be backported
to all stable versions up to 1.6.
This commit is contained in:
parent
0898c2d2f2
commit
79512b6d8f
@ -1184,6 +1184,9 @@ static int smp_fetch_proto_http(const struct arg *args, struct sample *smp, cons
|
||||
/* return a valid test if the current request is the first one on the connection */
|
||||
static int smp_fetch_http_first_req(const struct arg *args, struct sample *smp, const char *kw, void *private)
|
||||
{
|
||||
if (!smp->strm)
|
||||
return 0;
|
||||
|
||||
smp->data.type = SMP_T_BOOL;
|
||||
smp->data.u.sint = !(smp->strm->txn->flags & TX_NOT_FIRST);
|
||||
return 1;
|
||||
|
||||
Loading…
x
Reference in New Issue
Block a user