`sizeof(char)` is always 1, so it's not really needed.

But anyway, I'd recommend using `std::vector` or a similar data type that manages the memory for you. The reason is because if line 426 (`value = buff;`) (or any other line before `free(buff)`)throws an exception, the memory will be leaked.