awesomized / libmemcached

Resurrection of libmemcached
https://awesomized.github.io/libmemcached/
BSD 3-Clause "New" or "Revised" License
45 stars 26 forks source link

stack overflow in memcached_fetch_result #53

Closed m6w6 closed 4 years ago

m6w6 commented 4 years ago

Imported from Launchpad using lp2gh.


In the binary protocol, we occasionally do a lot of ADDs without any GETs. Many of the ADDs already have values, and so return error messages. In that case, we end up with 7000+ error responses on the wire, and the recursive call in binary_read_one_response produces a stack overflow. It's trivial to fix with the included patch.

m6w6 commented 4 years ago