Hi On Thu, Jun 4, 2015 at 12:39 PM, Sergei Zviagintsev <[email protected]> wrote: > sizeof(((_t *)0)->_m) -> FIELD_SIZEOF(_t, _m) > > Use conventional macro according to chapter 17 of > Documentation/CodingStyle. > > Signed-off-by: Sergei Zviagintsev <[email protected]> > --- > ipc/kdbus/util.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-)
Reviewed-by: David Herrmann <[email protected]> Thanks David > diff --git a/ipc/kdbus/util.h b/ipc/kdbus/util.h > index 9fedf8ab41cd..529716669fe7 100644 > --- a/ipc/kdbus/util.h > +++ b/ipc/kdbus/util.h > @@ -40,7 +40,7 @@ > ({ \ > u64 __user *_sz = \ > (void __user *)((u8 __user *)(_b) + offsetof(_t, _m)); \ > - copy_to_user(_sz, _s, sizeof(((_t *)0)->_m)); \ > + copy_to_user(_sz, _s, FIELD_SIZEOF(_t, _m)); \ > }) > > /** > -- > 1.8.3.1 > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
