msgpack: NULL terminate incoming strings

It's a 1-byte loss of memory but it allows us to skip copying and
NULL-terminating strings when interacting with vim functions that accept C
strings. This lowers the pressure on the allocator and saves lines of code
(no more dup/free pairs).

1 files changed, 1 insertions, 1 deletions

diff --git a/src/nvim/os/msgpack_rpc.c b/src/nvim/os/msgpack_rpc.c
index 423c5d584d..932a7717fd 100644
--- a/src/nvim/os/msgpack_rpc.c
+++ b/src/nvim/os/msgpack_rpc.c
@@ -163,7 +163,7 @@ bool msgpack_rpc_to_string(msgpack_object *obj, String *arg)
     return false;
   }
 
-  arg->data = xmemdup(obj->via.raw.ptr, obj->via.raw.size);
+  arg->data = xmemdupz(obj->via.raw.ptr, obj->via.raw.size);
   arg->size = obj->via.raw.size;
   return true;
 }