Handle 'SLAVEOF NO ONE' from master without segfaulting #955

Open
wants to merge 1 commit into
from

Conversation

Projects
None yet
1 participant

wallrat commented Feb 18, 2013

Patch for issue #954

Changes slaveofCommand to set flag REDIS_CLOSE_AFTER_REPLY instead of freeing client
immediately when client is the replication master. Allows master to send 'SLAVEOF NO ONE'
without slave crashing with a segfault.

@wallrat wallrat Handle 'SLAVEOF NO ONE' from master without segfaulting
Changes slaveofCommand to set flag REDIS_CLOSE_AFTER_REPLY instead of freeing client
immediately when client is the replication master. Allows master to send 'SLAVEOF NO ONE'
without slave crashing with a segfault.
116fcdc
Owner

wallrat commented on 116fcdc Feb 18, 2013

Passes tests and seems to work fine in our production env.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment