Re: [PATCH] netlink: Allow NLM_F_DUMP on netlink requests

[Denis Kenzior <denkenz@gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 863 bytes --]

Hi Patrik,

On 09/24/2014 04:13 AM, Patrik Flykt wrote:
> NLM_F_DUMP needs to be specified in cases where all existing network
> interfaces are requested. As NLM_F_DUMP happens to be covered by the
> current check for valid flags, update the check to allow the flag
> being set by the caller.
> ---
>   ell/netlink.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/ell/netlink.c b/ell/netlink.c
> index d247173..79a2a00 100644
> --- a/ell/netlink.c
> +++ b/ell/netlink.c
> @@ -396,7 +396,7 @@ LIB_EXPORT unsigned int l_netlink_send(struct l_netlink *netlink,
>   						!netlink->command_lookup)
>   		return 0;
>
> -	if (flags & 0xff)
> +	if (flags & 0xff & ~NLM_F_DUMP)
>   		return 0;

Maybe a nitpick, but should this be simply:

if (flags & ~NLM_F_DUMP)

?

>
>   	if (function)
>

Regards,
-Denis