Hi Felipe,
On Thu, Mar 16, 2017 at 05:01:41PM -0300, Felipe Guerreiro Barbosa Ruiz wrote:
> This fixes a regression introduced in d7bdcb874bcb, that removed the
> ability to use req.payload(0,0) to read the whole buffer content. The
> offending commit is present starting in version 1.6, so the patch
> should be backported to versions 1.6 and 1.7.
All useful indications are provided, that's perfect, I've applied it now.
Thank you very much Felipe. BTW, just for your information, the patch was
mangled by your mailer (lines were wrapped) but since it was easy I could
fix it by hand :
> diff --git a/src/payload.c b/src/payload.c
> index a02a8696..b80a19c9 100644
> --- a/src/payload.c
> +++ b/src/payload.c
> @@ -838,7 +838,7 @@ smp_fetch_payload(const struct arg *arg_p, struct
> sample *smp, const char *kw, v
> return 0;
>
> chn = ((smp->opt & SMP_OPT_DIR) == SMP_OPT_DIR_RES) ?
> &smp->strm->res : &smp->strm->req;
> - if (!buf_size || buf_size > global.tune.bufsize || buf_offset +
> buf_size > global.tune.bufsize) {
> + if (buf_size > global.tune.bufsize || buf_offset + buf_size >
> global.tune.bufsize) {
> /* will never match */
> smp->flags = 0;
> return 0;
Next time, in order to avoid this you can attach the patch to your mail,
mail agents generally don't touch attachments.
Thanks,
Willy
