Comment 40 for bug 1573594

Revision history for this message
Brian Aker (brianaker) wrote :

Please provide a test case clearly showing the bug in the upstream project.

Calling memset() would be incorrect in this case even if you were concerned about a NULL character ( i.e. the minimal change would be to insert a NULL character in the buffer after looked at the length of the returned string from memcached_response()... but you don't have to do that because it is already done for you ).

I mentioned the error path only because in that path there is no guarantee that a NULL would be added to the buffer, but then the buffer passed to memcached_response() goes unused.