| Message ID | 20230614120604.1196377-1-linma@zju.edu.cn (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | 44194cb1b6045dea33ae9a0d54fb7e7cd93a2e09 |
| Delegated to: | Netdev Maintainers |
| Headers | show |
| Series | [v2] net: tipc: resize nlattr array to correct size | expand |
Lin Ma <linma@zju.edu.cn> wrote: > According to nla_parse_nested_deprecated(), the tb[] is supposed to the > destination array with maxtype+1 elements. In current > tipc_nl_media_get() and __tipc_nl_media_set(), a larger array is used > which is unnecessary. This patch resize them to a proper size. Reviewed-by: Florian Westphal <fw@strlen.de>
>Subject: [PATCH v2] net: tipc: resize nlattr array to correct size > >According to nla_parse_nested_deprecated(), the tb[] is supposed to the >destination array with maxtype+1 elements. In current >tipc_nl_media_get() and __tipc_nl_media_set(), a larger array is used >which is unnecessary. This patch resize them to a proper size. > >Fixes: 1e55417d8fc6 ("tipc: add media set to new netlink api") >Fixes: 46f15c6794fb ("tipc: add media get/dump to new netlink api") >Signed-off-by: Lin Ma <linma@zju.edu.cn> >--- >V1 -> V2: add net in title, also add Fixes tag Reviewed-by: Tung Nguyen <tung.q.nguyen@dektech.com.au> > > net/tipc/bearer.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > >diff --git a/net/tipc/bearer.c b/net/tipc/bearer.c >index 53881406e200..cdcd2731860b 100644 >--- a/net/tipc/bearer.c >+++ b/net/tipc/bearer.c >@@ -1258,7 +1258,7 @@ int tipc_nl_media_get(struct sk_buff *skb, struct genl_info *info) > struct tipc_nl_msg msg; > struct tipc_media *media; > struct sk_buff *rep; >- struct nlattr *attrs[TIPC_NLA_BEARER_MAX + 1]; >+ struct nlattr *attrs[TIPC_NLA_MEDIA_MAX + 1]; > > if (!info->attrs[TIPC_NLA_MEDIA]) > return -EINVAL; >@@ -1307,7 +1307,7 @@ int __tipc_nl_media_set(struct sk_buff *skb, struct genl_info *info) > int err; > char *name; > struct tipc_media *m; >- struct nlattr *attrs[TIPC_NLA_BEARER_MAX + 1]; >+ struct nlattr *attrs[TIPC_NLA_MEDIA_MAX + 1]; > > if (!info->attrs[TIPC_NLA_MEDIA]) > return -EINVAL; >-- >2.17.1
Hello: This patch was applied to netdev/net.git (main) by Jakub Kicinski <kuba@kernel.org>: On Wed, 14 Jun 2023 20:06:04 +0800 you wrote: > According to nla_parse_nested_deprecated(), the tb[] is supposed to the > destination array with maxtype+1 elements. In current > tipc_nl_media_get() and __tipc_nl_media_set(), a larger array is used > which is unnecessary. This patch resize them to a proper size. > > Fixes: 1e55417d8fc6 ("tipc: add media set to new netlink api") > Fixes: 46f15c6794fb ("tipc: add media get/dump to new netlink api") > Signed-off-by: Lin Ma <linma@zju.edu.cn> > > [...] Here is the summary with links: - [v2] net: tipc: resize nlattr array to correct size https://git.kernel.org/netdev/net/c/44194cb1b604 You are awesome, thank you!
diff --git a/net/tipc/bearer.c b/net/tipc/bearer.c index 53881406e200..cdcd2731860b 100644 --- a/net/tipc/bearer.c +++ b/net/tipc/bearer.c @@ -1258,7 +1258,7 @@ int tipc_nl_media_get(struct sk_buff *skb, struct genl_info *info) struct tipc_nl_msg msg; struct tipc_media *media; struct sk_buff *rep; - struct nlattr *attrs[TIPC_NLA_BEARER_MAX + 1]; + struct nlattr *attrs[TIPC_NLA_MEDIA_MAX + 1]; if (!info->attrs[TIPC_NLA_MEDIA]) return -EINVAL; @@ -1307,7 +1307,7 @@ int __tipc_nl_media_set(struct sk_buff *skb, struct genl_info *info) int err; char *name; struct tipc_media *m; - struct nlattr *attrs[TIPC_NLA_BEARER_MAX + 1]; + struct nlattr *attrs[TIPC_NLA_MEDIA_MAX + 1]; if (!info->attrs[TIPC_NLA_MEDIA]) return -EINVAL;
According to nla_parse_nested_deprecated(), the tb[] is supposed to the destination array with maxtype+1 elements. In current tipc_nl_media_get() and __tipc_nl_media_set(), a larger array is used which is unnecessary. This patch resize them to a proper size. Fixes: 1e55417d8fc6 ("tipc: add media set to new netlink api") Fixes: 46f15c6794fb ("tipc: add media get/dump to new netlink api") Signed-off-by: Lin Ma <linma@zju.edu.cn> --- V1 -> V2: add net in title, also add Fixes tag net/tipc/bearer.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)