Skip to content

Commit fc9e81c

Browse files
committed
ANDROID: Fix up merge issues in 6.1.64 in net/netfilter/nf_tables_api.c
When merging 6.1.64 into the 'android14-6.1-lts' branch, which contained a subset of the upstream changes in nf_tables_api.c, the merge got confused and did it "backwards", backing out some of the needed fixes. This happens when we cherry-pick a subset of the upstream fixes into the 'android14-6.1' branch and then merge the LTS changes. Fixes: f1bc13c ("Merge 6.1.64 into android14-6.1-lts") Change-Id: I8f510160f4ca43a05be265173daccb962531b1b4 Signed-off-by: Greg Kroah-Hartman <gregkh@google.com>
1 parent 31e1ff2 commit fc9e81c

1 file changed

Lines changed: 3 additions & 14 deletions

File tree

net/netfilter/nf_tables_api.c

Lines changed: 3 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -9274,12 +9274,11 @@ void nft_trans_gc_queue_sync_done(struct nft_trans_gc *trans)
92749274
struct nft_trans_gc *nft_trans_gc_catchall_async(struct nft_trans_gc *gc,
92759275
unsigned int gc_seq)
92769276
{
9277-
struct nft_set_elem_catchall *catchall, *next;
9277+
struct nft_set_elem_catchall *catchall;
92789278
const struct nft_set *set = gc->set;
9279-
struct nft_elem_priv *elem_priv;
92809279
struct nft_set_ext *ext;
92819280

9282-
list_for_each_entry_safe(catchall, next, &set->catchall_list, list) {
9281+
list_for_each_entry_rcu(catchall, &set->catchall_list, list) {
92839282
ext = nft_set_elem_ext(set, catchall->elem);
92849283

92859284
if (!nft_set_elem_expired(ext))
@@ -9293,17 +9292,7 @@ struct nft_trans_gc *nft_trans_gc_catchall_async(struct nft_trans_gc *gc,
92939292
if (!gc)
92949293
return NULL;
92959294

9296-
elem_priv = catchall->elem;
9297-
if (sync) {
9298-
struct nft_set_elem elem = {
9299-
.priv = elem_priv,
9300-
};
9301-
9302-
nft_setelem_data_deactivate(gc->net, gc->set, &elem);
9303-
nft_setelem_catchall_destroy(catchall);
9304-
}
9305-
9306-
nft_trans_gc_elem_add(gc, elem_priv);
9295+
nft_trans_gc_elem_add(gc, catchall->elem);
93079296
}
93089297

93099298
return gc;

0 commit comments

Comments
 (0)