I have grumbled about the evilness of "current_base" before, and this just
proves the point: If two threads call event_init() at the same time, they will both stomp on the same memory via current_base, and in all likelyhood, neither of them will manage to get the job done properly. Instead work on a local variable and don't assign to current_base until the setup is complete. This should be submitted to Niels Provos git-svn-id: http://www.varnish-cache.org/svn/trunk/varnish-cache@288 d4fa192b-c00b-0410-8231-f00ffab90ce4
Showing
Please register or sign in to comment