On Tue, May 23, 2023 at 11:56:18AM +0200, Claudio Jeker wrote: > There is a lot of duplication in iked's imsg_util.c > Now here is a minimal diff removing all extra parts added that are not > needed anymore. The removal of ibuf_zero() depends partially on my > previous imsg diff. > > With my imsg diff allocation are always zeroed so no need to call > imsg_zero(). In ibuf_release() there is no need to call ibuf_zero() since > the imsg API is already using freezero(). Also the NULL check is not > needed. I moved the memset() into ibuf_reserve() so that can also go away. > > Iked does some rather unholy things with ibufs which I want to slowly > clean up. This is step 1.
ok tb