[zeromq-dev] nmap patch for github issues 147+149 (nmap)
Pieter Hintjens
ph at imatix.com
Mon Jan 17 10:19:03 CET 2011
Hi Thijs,
This is really great, thanks! Could you read the page at
http://www.zeromq.org/docs:contributing and resubmit the patch as a
signed-off patch following those instructions?
Cheers
Pieter
On Mon, Jan 17, 2011 at 10:04 AM, Thijs Terlouw <thijsterlouw at gmail.com> wrote:
> Hi Martin,
>
> This patch solves the crashes I was seeing when I ran nmap against a
> range of listening ZeroMQ sockets (github issues 147 + 149)
> I'm not sure if I patched it correctly (for example the want_terminate
> boolean in zmq_engine_t is a bit of a hack), but at least this works
> for me and in the worst case it can give you some ideas how to solve
> it correctly.
>
> There were two seperate issues:
> 1. zmq_engine_t::error() assumed it was plugged in
> - it's appears to be valid to be in an unplugged state and call the
> error() function in the in_event() function
>
> 2. zmq_engine_t::error() deleted itself which let to dangling pointer
> and virtual function call in session_t
> - zmq_init_t dispatches the ephemeral engine
> - zmq_init_t send_attach()
> - ephemeral engine encounters an error, deletes itself
> - session_t process_attach tries to plug the (already destructed)
> engine back in
>
> if you want to use the patch, it's supplied under LGPL :)
>
> Thijs
>
> _______________________________________________
> zeromq-dev mailing list
> zeromq-dev at lists.zeromq.org
> http://lists.zeromq.org/mailman/listinfo/zeromq-dev
>
>
More information about the zeromq-dev
mailing list