Subject: Re: [patch net-next v3 8/9] devlink: add a command to set notification filter and use it for multicasts

From: Jakub Kicinski <[email-masked]>
Date: Mon, 20 Nov 2023 18:51:23 -0800

On Mon, 20 Nov 2023 09:46:56 +0100 Jiri Pirko wrote:
> +	if (attrs[DEVLINK_ATTR_BUS_NAME])
> +		data_size += nla_len(attrs[DEVLINK_ATTR_BUS_NAME]) + 1;
> +	if (attrs[DEVLINK_ATTR_DEV_NAME])
> +		data_size += nla_len(attrs[DEVLINK_ATTR_DEV_NAME]) + 1;

> +	/* Don't attach empty filter. */
> +	if (!flt->bus_name && !flt->dev_name) {

Should the attrs be checked with GENL_REQ_ATTR_CHECK() then?