Details
Description
The array's returned by getMulti() are not correctly keyed or are in some other way broken. When one tries to reference the key via it's name, an "undefined index" error is returned. array_key_exists() also returns false.
This paste has some sample code:
https://friendpaste.com/MJS2O8G5b7kRqslp2pMBo
Both the top and bottom cases should end in bool(true).
avsej mentioned in IRC that these two portions of code should be compared:
https://github.com/couchbase/php-ext-couchbase/blob/master/couchbase.c#L940
https://github.com/php-memcached-dev/php-memcached/blob/master/php_memcached.c#L828
Thanks for digging into this issue.
Attachments
For Gerrit Dashboard: PCBC-66 | ||||||
---|---|---|---|---|---|---|
# | Subject | Branch | Project | Status | CR | V |
16144,6 | PCBC-66: null terminate keys in getMulti responses | master | php-ext-couchbase | Status: MERGED | +2 | +1 |
16859,1 | PCBC-66: null terminate keys in getMulti responses | 1.0.x | php-ext-couchbase | Status: MERGED | +2 | +1 |
19942,1 | Merge branch '1.0.x' | master | php-ext-couchbase | Status: MERGED | +2 | +1 |
20449,2 | PCBC-66 revisited | master | php-ext-couchbase | Status: MERGED | +2 | +1 |
20629,4 | PCBC-66 revisited: fix mget/mset (backport) | 1.0.x | php-ext-couchbase | Status: MERGED | +2 | +1 |
20864,2 | Fix for PCBC-115, getStats() array keys malformed | 1.0.x | php-ext-couchbase | Status: MERGED | +2 | +1 |
20875,3 | PCBC-108 / PCBC-115: version changes, stat array key fixes | master | php-ext-couchbase | Status: MERGED | +2 | +1 |