[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <172061821475.5582.9226948763101271068@kwain.local>
Date: Wed, 10 Jul 2024 15:30:14 +0200
From: Antoine Tenart <atenart@...nel.org>
To: Adrian Moreno <amorenoz@...hat.com>, netdev@...r.kernel.org
Cc: Adrian Moreno <amorenoz@...hat.com>, Yotam Gigi <yotam.gi@...il.com>, David S. Miller <davem@...emloft.net>, Eric Dumazet <edumazet@...gle.com>, Jakub Kicinski <kuba@...nel.org>, Paolo Abeni <pabeni@...hat.com>, Ido Schimmel <idosch@...dia.com>, Eelco Chaudron <echaudro@...hat.com>, Aaron Conole <aconole@...hat.com>, linux-kernel@...r.kernel.org
Subject: Re: [PATCH net-next v2] net: psample: fix flag being set in wrong skb
Hi Adrián,
Quoting Adrian Moreno (2024-07-10 11:07:42)
> A typo makes PSAMPLE_ATTR_SAMPLE_RATE netlink flag be added to the wrong
> sk_buff.
>
> Fix the error and make the input sk_buff pointer "const" so that it
> doesn't happen again.
>
> Also modify OVS psample test to verify the flag is properly emitted.
I don't see that part; although it can be sent as a follow-up and not
part of the fix.
Thanks,
Antoine
> Fixes: 7b1b2b60c63f ("net: psample: allow using rate as probability")
> Signed-off-by: Adrian Moreno <amorenoz@...hat.com>
> ---
> include/net/psample.h | 8 +++++---
> net/psample/psample.c | 7 ++++---
> 2 files changed, 9 insertions(+), 6 deletions(-)
>
> diff --git a/include/net/psample.h b/include/net/psample.h
> index c52e9ebd88dd..5071b5fc2b59 100644
> --- a/include/net/psample.h
> +++ b/include/net/psample.h
> @@ -38,13 +38,15 @@ struct sk_buff;
>
> #if IS_ENABLED(CONFIG_PSAMPLE)
>
> -void psample_sample_packet(struct psample_group *group, struct sk_buff *skb,
> - u32 sample_rate, const struct psample_metadata *md);
> +void psample_sample_packet(struct psample_group *group,
> + const struct sk_buff *skb, u32 sample_rate,
> + const struct psample_metadata *md);
>
> #else
>
> static inline void psample_sample_packet(struct psample_group *group,
> - struct sk_buff *skb, u32 sample_rate,
> + const struct sk_buff *skb,
> + u32 sample_rate,
> const struct psample_metadata *md)
> {
> }
> diff --git a/net/psample/psample.c b/net/psample/psample.c
> index f48b5b9cd409..a0ddae8a65f9 100644
> --- a/net/psample/psample.c
> +++ b/net/psample/psample.c
> @@ -360,8 +360,9 @@ static int psample_tunnel_meta_len(struct ip_tunnel_info *tun_info)
> }
> #endif
>
> -void psample_sample_packet(struct psample_group *group, struct sk_buff *skb,
> - u32 sample_rate, const struct psample_metadata *md)
> +void psample_sample_packet(struct psample_group *group,
> + const struct sk_buff *skb, u32 sample_rate,
> + const struct psample_metadata *md)
> {
> ktime_t tstamp = ktime_get_real();
> int out_ifindex = md->out_ifindex;
> @@ -498,7 +499,7 @@ void psample_sample_packet(struct psample_group *group, struct sk_buff *skb,
> goto error;
>
> if (md->rate_as_probability)
> - nla_put_flag(skb, PSAMPLE_ATTR_SAMPLE_PROBABILITY);
> + nla_put_flag(nl_skb, PSAMPLE_ATTR_SAMPLE_PROBABILITY);
>
> genlmsg_end(nl_skb, data);
> genlmsg_multicast_netns(&psample_nl_family, group->net, nl_skb, 0,
> --
> 2.45.2
>
>
Powered by blists - more mailing lists