~vcs-imports/memcached/1.4

Viewing all changes in revision 1403.

  • Committer: dormando
  • Date: 2018-03-28 02:16:18 UTC
  • Revision ID: git-v1:95e6469bd2ceef92bcaaf140e2724fc73d556185
extstore: fix ref leak when using binprot GATK

GATK returns a key but not the value. c->io_wraplist is only appended if the
value is to be returned, but c->item is skipped if it is an ITEM_HDR at all.

This now checks for the ITEM_HDR bit being set but also !value
which then reclaims the reference normally.

I knew doubling up the cleanup code made it a lot more complex, and hope to
flatten that to a single path. Also the TOUCH/GAT/GATK binprot code has no real
test coverage, nor mc-crusher entries. Should be worth fixing.

expand all expand all

Show diffs side-by-side

added added

removed removed

Lines of Context: