Message ID | 20221028132943.304ECC433B5@smtp.kernel.org (mailing list archive) |
---|---|
State | Not Applicable |
Delegated to: | Kalle Valo |
Headers | show |
Series | pull-request: wireless-next-2022-10-28 | expand |
Kalle Valo <kvalo@kernel.org> writes: > Note: wireless tree was merged[1] to wireless-next to avoid some > conflicts with mac80211 patches between the trees. Unfortunately there > are still two smaller conflicts in net/mac80211/util.c which Stephen > also reported[2]. In the first conflict initialise scratch_len to > "params->scratch_len ?: 3 * params->len" (note number 3, not 2!) and > in the second conflict take the version which uses elems->scratch_pos. > > Git diff output should like this: > > --- a/net/mac80211/util.c > +++ b/net/mac80211/util.c > @@@ -1506,7 -1648,7 +1650,7 @@@ ieee802_11_parse_elems_full(struct ieee > const struct element *non_inherit = NULL; > u8 *nontransmitted_profile; > int nontransmitted_profile_len = 0; > - size_t scratch_len = params->len; > - size_t scratch_len = params->scratch_len ?: 2 * params->len; > ++ size_t scratch_len = params->scratch_len ?: 3 * params->len; > > elems = kzalloc(sizeof(*elems) + scratch_len, GFP_ATOMIC); > if (!elems) And in case want to see the final result, here's an example merge I pushed to our pending branch: https://git.kernel.org/pub/scm/linux/kernel/git/wireless/wireless-next.git/tree/net/mac80211/util.c?h=pending&id=0879f594289e36546974c17f10bf587d9303e724#n1646
On Fri, 28 Oct 2022 13:29:43 +0000 (UTC) Kalle Valo wrote: > Note: wireless tree was merged[1] to wireless-next to avoid some > conflicts with mac80211 patches between the trees. Unfortunately there > are still two smaller conflicts in net/mac80211/util.c which Stephen > also reported[2]. In the first conflict initialise scratch_len to > "params->scratch_len ?: 3 * params->len" (note number 3, not 2!) and > in the second conflict take the version which uses elems->scratch_pos. > > Git diff output should like this: > > --- a/net/mac80211/util.c > +++ b/net/mac80211/util.c > @@@ -1506,7 -1648,7 +1650,7 @@@ ieee802_11_parse_elems_full(struct ieee > const struct element *non_inherit = NULL; > u8 *nontransmitted_profile; > int nontransmitted_profile_len = 0; > - size_t scratch_len = params->len; > - size_t scratch_len = params->scratch_len ?: 2 * params->len; > ++ size_t scratch_len = params->scratch_len ?: 3 * params->len; > > elems = kzalloc(sizeof(*elems) + scratch_len, GFP_ATOMIC); > if (!elems) > > [1] https://git.kernel.org/pub/scm/linux/kernel/git/wireless/wireless-next.git/commit/?id=dfd2d876b3fda1790bc0239ba4c6967e25d16e91 > [2] https://lore.kernel.org/all/20221020032340.5cf101c0@canb.auug.org.au/ Thanks! I only saw one conflict FWIW
Hello: This pull request was applied to netdev/net-next.git (master) by Jakub Kicinski <kuba@kernel.org>: On Fri, 28 Oct 2022 13:29:43 +0000 (UTC) you wrote: > Hi, > > here's a pull request to net-next tree, more info below. Please let me know if > there are any problems. > > Kalle > > [...] Here is the summary with links: - pull-request: wireless-next-2022-10-28 https://git.kernel.org/netdev/net-next/c/196dd92a00ff You are awesome, thank you!
On Fri, 2022-10-28 at 18:34 -0700, Jakub Kicinski wrote: > > > > --- a/net/mac80211/util.c > > +++ b/net/mac80211/util.c > > @@@ -1506,7 -1648,7 +1650,7 @@@ ieee802_11_parse_elems_full(struct ieee > > const struct element *non_inherit = NULL; > > u8 *nontransmitted_profile; > > int nontransmitted_profile_len = 0; > > - size_t scratch_len = params->len; > > - size_t scratch_len = params->scratch_len ?: 2 * params->len; > > ++ size_t scratch_len = params->scratch_len ?: 3 * params->len; > > > > elems = kzalloc(sizeof(*elems) + scratch_len, GFP_ATOMIC); > > if (!elems) > > > > [1] https://git.kernel.org/pub/scm/linux/kernel/git/wireless/wireless-next.git/commit/?id=dfd2d876b3fda1790bc0239ba4c6967e25d16e91 > > [2] https://lore.kernel.org/all/20221020032340.5cf101c0@canb.auug.org.au/ > > Thanks! I only saw one conflict FWIW > Hah. Me too, when I tried this to see what the resolution should be. Git versions or something? johannes