On Mon, Jun 12, 2023 at 12:18:19PM +0200, Theo Buehler wrote: > On Mon, Jun 12, 2023 at 11:43:11AM +0200, Claudio Jeker wrote: > > To generate the IMSG_CTL_SHOW_RIB_ATTR message just use attr_writebuf() > > instead of going via ibuf_reserve() and attr_write(). > > > > While there make a pointer != NULL check more obvious. > > ok > > > @@ -2934,12 +2933,8 @@ rde_dump_rib_as(struct prefix *p, struct > > IMSG_CTL_SHOW_RIB_ATTR, 0, pid, > > attr_optlen(a))) == NULL) > > This is the last use of attr_optlen(). I don't mind it, but it might as > well be inlined.
Or just removed and replaced with 0. imsg_create() does not really care about the lenght (apart from extra length checks). I can funnel that into the same diff or do it as a 2nd step. -- :wq Claudio