Cc'ing netfilter-devel. On Tue, Jun 14, 2016 at 07:39:27PM +0530, Kishan Sandeep wrote: > + netdev > > On Sat, Jun 11, 2016 at 10:18 AM, Kishan Sandeep > <sandeepkishan108@xxxxxxxxx> wrote: > > strncpy generally perferable fo non-terminated > > fixed-width strings. For NULL termination strlcpy > > is preferrable. > > > > Signed-off-by: Kishan Sandeep <sandeepkishan108@xxxxxxxxx> > > --- > > net/netfilter/xt_repldata.h | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/net/netfilter/xt_repldata.h b/net/netfilter/xt_repldata.h > > index 8fd3241..a460211 100644 > > --- a/net/netfilter/xt_repldata.h > > +++ b/net/netfilter/xt_repldata.h > > @@ -28,7 +28,7 @@ > > if (tbl == NULL) \ > > return NULL; \ > > term = (struct type##_error *)&(((char *)tbl)[term_offset]); \ > > - strncpy(tbl->repl.name, info->name, sizeof(tbl->repl.name)); \ > > + strlcpy(tbl->repl.name, info->name, sizeof(tbl->repl.name)); \ I don't think this is actually fixing anything. Tables in x_tables have a known and fixed name that is defined from the kernel side, that is always smaller that the buffer we have there. So are you observing any real problem from there? Thanks. -- To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
