Stable updates. Refcount leak + SET change on alloc fail
dormando at rydia.net
Sun Jun 29 23:58:10 UTC 2008
git-send-email is being a punk, so please check the three latest commits
(can also clone the repo:
git clone http://consoleninja.net/code/dpm/dpm.git and fetch my 'stable'
This is first in a small series of patches going in before 1.2.6-rc1.
I've audited all item_get* calls and ensured they were matched with
item_remove() calls properly. I might've missed something, so feel free
to check for yourselves :)
The other change is a little weird, but makes sense to me. If you 'set'
an item and the set fails alloc (too large, OOM), unlink an item if it
already exists. This would help in the case of someone getting OOM
errors, or if they're sloppy and occasionally fail to update their
cache. Giving into the side of additional cache misses for data correctness.
Please review, or if you hate gitweb whine and I'll make git-send-email
I'm going through the stack for anything else to apply to stable. If
you've sent a relatively minor patch and would totally love to see it in
1.2.6, now's the time to speak up.
More information about the memcached