Updates:
Status: Started
Owner: [email protected]
Comment #8 on issue 106 by [email protected]: binary protocol parsing can
cause memcached server lockup
http://code.google.com/p/memcached/issues/detail?id=106
I just pulled a patch similar to this into my 1.4.7 tree.
This was confusing me a bit... the extra recvfrom seemed pointless and it'd
hang while running. The test from #158 would spin out of control without
the recvfrom.
But the initial state of a UDP connection is actually conn_read, not
conn_new_cmd. Using the correct state makes it find the next UDP packet and
throw it away properly.
Still iffy on the rbytes bit, but so long as multiple command parsing works
(which I don't think it does now anyway), that should be correct.
Punting to trond in case he wants to port this properly to 1.6.0, or take a
look at least. I'll be pushing my for_147 branch to github at some point.
