On Tue, Jan 31, 2023 at 12:47:03PM +0100, Paolo Abeni wrote: > Hello, > > On Sun, 2023-01-29 at 18:51 +0200, Leon Romanovsky wrote: > > In netdev common pattern, extack pointer is forwarded to the drivers > > to be filled with error message. However, the caller can easily > > overwrite the filled message. > > > > Instead of adding multiple "if (!extack->_msg)" checks before any > > NL_SET_ERR_MSG() call, which appears after call to the driver, let's > > add new macro to common code. > > > > [1] https://lore.kernel.org/all/Y9Irgrgf3uxOjwUm@unreal > > Signed-off-by: Leon Romanovsky <leonro@xxxxxxxxxx> > > I'm sorry for nit-picking, but checkpatch complains the author > (leon@xxxxxxxxxx) does not match the SoB tag. A v3 with a suitable > From: tag should fix that. Sure, will resend. > > Thanks, > > Paolo >