Rewrite the human-readable protocol parser to avoid memory leaks

[danielealbano]

The current implementation of the human-readable Redis protocol parser is buggy as it searches for a space and then perform a number of additional operations expecting data to be present afterwards.

The bug is being triggered by the code at line

cachegrand/src/protocol/redis/protocol_redis_reader.c

Line 175 in 69c5e92

} while (arg_start_char_ptr++ && arg_start_char_ptr < buffer_end_ptr);

The test

cachegrand/tests/protocols/redis/test-protocol-redis-reader-inline.cpp

Line 510 in 69c5e92

// SECTION("multiple argument 1 byte at time, no quotes, with new line, clone data") {

triggers the issue