Julien Semaan

Results 149 comments of Julien Semaan

I'm not so expert in C (not to say I'm bad) I'll give it a shot but I'm not too confident I'll get it

Seems part of it is in http_request_free where not everthing is freed properly ``` diff --git a/src/http.c b/src/http.c index f571e5b..52b78de 100644 --- a/src/http.c +++ b/src/http.c @@ -237,6 +237,20 @@ http_request_free(request_t...

Much much better with that commit. Still seems that the flow leak is there. There is memory not freed with mallocs from http_new Here is the full valgrind report :...

Any progress on that ?

New pastebin link : http://pastebin.com/qdTQ91cV

What do we do about this, its been sitting here for a long time. Either we get this in for 11.1 or lets just clear the milestone

@nqb and @jrouzierinverse, what prevents this from being merged?

Well the errors are sockets being backed up in /usr/local/pf/var/run No need to backup the content of that directory as it contains PID files and sockets that can't be restored...

The var/run isn't part of the tar anymore so the sockets message should be gone. I still believe we should keep the remaining of the output so we know what...

Lets give some love to that script in 8.1