Stable updates. Refcount leak + SET change on alloc fail

dormando dormando at rydia.net
Sun Jun 29 23:59:43 UTC 2008


Uh, I'm an idiot. that's the DPM repo.

git clone http://consoleninja.net/code/memcached/memcached.git - sigh.

dormando wrote:
> Yo,
> 
> git-send-email is being a punk, so please check the three latest commits
> here:
> 
> http://consoleninja.net/gitweb/gitweb.cgi?p=memcached.git;a=shortlog;h=stable
> 
> (can also clone the repo:
> git clone http://consoleninja.net/code/dpm/dpm.git and fetch my 'stable'
> branch).
> 
> 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
> cooperate again.
> 
> 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.
> 
> -Dormando



More information about the memcached mailing list