like that ?

--- src/proxy.c.orig    2016-06-07 00:56:29.001214621 +0500
+++ src/proxy.c    2016-06-07 00:59:01.823568543 +0500
@@ -426,11 +426,14 @@
             hdr->index = curpx->nb_req_cap++;
             curpx->req_cap = hdr;
         }
-        if (strcmp(args[2], "response") == 0) {
+        else if (strcmp(args[2], "response") == 0) {
             hdr->next = curpx->rsp_cap;
             hdr->index = curpx->nb_rsp_cap++;
             curpx->rsp_cap = hdr;
         }
+        else {
+            free(hdr);
+        }
         return 0;
     }
     else {


2016-06-07 0:34 GMT+05:00 Sasha Pachev <sa...@asksasha.com>:

> I took a quick look. The first one I looked at was a minor issue, but
> still legitimate.
>
> http://chipitsine.github.io/haproxy-1.7-dev/report-0bb120.html#EndPath
>
> line 419
>
> hdr should be freed if args[2] is neither "request" nor "response",
> otherwise the allocated address is permanently forgotten.
>
>
> On Mon, Jun 6, 2016 at 12:52 PM, Maciej Katafiasz
> <mkatafi...@purestorage.com> wrote:
> > On 6 June 2016 at 09:41, Илья Шипицин <chipits...@gmail.com> wrote:
> >> also, is there some bug tracker ? CI (like travis-ci or jenkins) ?
> >
> > No, this list is the place to report things.
> >
> > Cheers,
> > Maciej
> >
>
>
>
> --
> Sasha Pachev
>
> Fast Running Blog.
> http://fastrunningblog.com
> Run. Blog. Improve. Repeat.
>

Reply via email to