-
Notifications
You must be signed in to change notification settings - Fork 27
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
error after resetting servers and trying to add new ones #13
Comments
Just to add some more - I'm using persistent memcached connenction |
Thanks, I am looking into this. |
test for this issue |
Fix + test for this issue attached |
cool, latest patch works fine for 1.0.18, tested in production |
works for us as well |
=dev-libs/libmemcached-1.0.18-r2 segfaults with the OPT_LIBKETAMA_COMPATIBLE enabled and a call to addServer, then resetServerList and another call to addServer. Reproducible: Always Steps to Reproduce: Actual Results: Expected Results: |
Created attachment 400200 Patch from the launchpad bug report. |
Please upload your emerge --info. |
emerge --info
|
fixed in 1.0.18-r3 |
Would it be possible to apply this patch to mainline libmemcached in a 1.0.19 release? This is affecting us too, running CentOS 7. |
Imported from Launchpad using lp2gh.
I'm using PHP memcached extension (master branch of git://github.com/php-memcached-dev/php-memcached.git) . When compiled with 1.0.15 and resetting servers list and reading new servers works fine, but when compiled with 1.0.16 PHP dies without errors or stack trace :(
resetServers uses:
memcached_servers_reset(m_obj->memc);
addServers uses:
memcached_server_list_append_with_weight(list, host, port, weight, &status);
memcached_server_push(m_obj->memc, list);
memcached_server_list_free(list);
I didn't have time to debug any further ... sorry
The text was updated successfully, but these errors were encountered: