Avoid calling malloc with a zero length argument.
authorclaudio <claudio@openbsd.org>
Tue, 23 May 2023 12:41:28 +0000 (12:41 +0000)
committerclaudio <claudio@openbsd.org>
Tue, 23 May 2023 12:41:28 +0000 (12:41 +0000)
commite2fc702335c2e6aa08781bd346ec98c392615337
tree3256df3218b9f1742811d5d85661f1b93087c768
parentfcb77b6b49ad76dea8c856e3a3a8cbc7f614d1d2
Avoid calling malloc with a zero length argument.

ibuf_open() will return an error in this case while ibuf_dynamic() accepts
a 0 len argument and just initialized the buffer and length to zero.
A later ibuf_realloc() call will take care of allocating the buffer.

Additionally switch from malloc() to calloc() when allocating the buffer
this way the buffer is initalized and in ibuf_reserve() an addtional
memset() is used to make sure that the reserved data is zeroed.

OK tb@
lib/libutil/imsg-buffer.c