On Fri, May 25, 2018 at 09:29:35AM +0200, Jesper Dangaard Brouer wrote: > > On Thu, 24 May 2018 23:37:45 +0300 Julian Anastasov <j...@ssi.bg> wrote: > > > Size or length? Here is the answer: > > > > - IP_VS_SCHEDNAME_MAXLEN and IP_VS_IFNAME_MAXLEN are sizes > > because they are used in kernel structures exported to user > > space for the old setsockopt interface. We can not change > > these structures in the kernel. > > > > - IP_VS_PENAME_MAXLEN and IP_VS_PEDATA_MAXLEN are max lengths > > because they are not exported to the old interface. > > > > As result: > > - buffers should have space for NUL terminator > > - strncpy should use sizeof(buffer) - 1 as max length > > > > As we change the libipvs structures, their users should be > > recompiled. > > > > Signed-off-by: Julian Anastasov <j...@ssi.bg> > > This all looks fine to me. I'll give other people a little time to > review and ACK, before I apply this. > > (To Julian) did you find this by manual review, or did you use some tool > to find these?
Acked-by: Simon Horman <ho...@verge.net.au> _______________________________________________ Please read the documentation before posting - it's available at: http://www.linuxvirtualserver.org/ LinuxVirtualServer.org mailing list - lvs-users@LinuxVirtualServer.org Send requests to lvs-users-requ...@linuxvirtualserver.org or go to http://lists.graemef.net/mailman/listinfo/lvs-users
