Florian Westphal 960cf4f812 netfilter: nf_tables: reject QUEUE/DROP verdict parameters ...

commit f342de4e2f upstream.

This reverts commit e0abdadcc6.

core.c:nf_hook_slow assumes that the upper 16 bits of NF_DROP
verdicts contain a valid errno, i.e. -EPERM, -EHOSTUNREACH or similar,
or 0.

Due to the reverted commit, its possible to provide a positive
value, e.g. NF_ACCEPT (1), which results in use-after-free.

Its not clear to me why this commit was made.

NF_QUEUE is not used by nftables; "queue" rules in nftables
will result in use of "nft_queue" expression.

If we later need to allow specifiying errno values from userspace
(do not know why), this has to call NF_DROP_GETERR and check that
"err <= 0" holds true.

Fixes: e0abdadcc6 ("netfilter: nf_tables: accept QUEUE/DROP verdict parameters")
Cc: stable@vger.kernel.org
Reported-by: Notselwyn <notselwyn@pwning.tech>
Signed-off-by: Florian Westphal <fw@strlen.de>
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

2024-02-23 08:54:32 +01:00

..

ipset

…

ipvs

ipvs: avoid stat macros calls from preemptible context

2024-01-25 14:52:54 -08:00

core.c

…

Kconfig

…

Makefile

…

nf_conncount.c

…

nf_conntrack_acct.c

…

nf_conntrack_amanda.c

…

nf_conntrack_broadcast.c

…

nf_conntrack_core.c

…

nf_conntrack_ecache.c

…

nf_conntrack_expect.c

…

nf_conntrack_extend.c

…

nf_conntrack_ftp.c

…

nf_conntrack_h323_asn1.c

…

nf_conntrack_h323_main.c

…

nf_conntrack_h323_types.c

…

nf_conntrack_helper.c

…

nf_conntrack_irc.c

…

nf_conntrack_labels.c

…

nf_conntrack_netbios_ns.c

…

nf_conntrack_netlink.c

…

nf_conntrack_pptp.c

…

nf_conntrack_proto_dccp.c

…

nf_conntrack_proto_generic.c

…

nf_conntrack_proto_gre.c

…

nf_conntrack_proto_icmp.c

…

nf_conntrack_proto_icmpv6.c

…

nf_conntrack_proto_sctp.c

…

nf_conntrack_proto_tcp.c

…

nf_conntrack_proto_udp.c

…

nf_conntrack_proto.c

…

nf_conntrack_sane.c

…

nf_conntrack_seqadj.c

…

nf_conntrack_sip.c

…

nf_conntrack_snmp.c

…

nf_conntrack_standalone.c

…

nf_conntrack_tftp.c

…

nf_conntrack_timeout.c

…

nf_conntrack_timestamp.c

…

nf_dup_netdev.c

…

nf_flow_table_core.c

…

nf_flow_table_inet.c

…

nf_flow_table_ip.c

…

nf_flow_table_offload.c

…

nf_hooks_lwtunnel.c

…

nf_internals.h

…

nf_log_syslog.c

…

nf_log.c

…

nf_nat_amanda.c

…

nf_nat_core.c

…

nf_nat_ftp.c

…

nf_nat_helper.c

…

nf_nat_irc.c

…

nf_nat_masquerade.c

…

nf_nat_proto.c

…

nf_nat_redirect.c

…

nf_nat_sip.c

…

nf_nat_tftp.c

…

nf_queue.c

…

nf_sockopt.c

…

nf_synproxy_core.c

…

nf_tables_api.c

netfilter: nf_tables: reject QUEUE/DROP verdict parameters

2024-02-23 08:54:32 +01:00

nf_tables_core.c

…

nf_tables_offload.c

…

nf_tables_trace.c

…

nfnetlink_acct.c

…

nfnetlink_cthelper.c

…

nfnetlink_cttimeout.c

…

nfnetlink_hook.c

…

nfnetlink_log.c

…

nfnetlink_osf.c

…

nfnetlink_queue.c

…

nfnetlink.c

…

nft_bitwise.c

…

nft_byteorder.c

…

nft_chain_filter.c

netfilter: nft_chain_filter: handle NETDEV_UNREGISTER for inet/ingress basechain

2024-02-23 08:54:32 +01:00

nft_chain_nat.c

…

nft_chain_route.c

…

nft_cmp.c

…

nft_compat.c

netfilter: nf_tables: validate NFPROTO_* family

2024-02-23 08:54:29 +01:00

nft_connlimit.c

netfilter: nft_connlimit: memleak if nf_ct_netns_get() fails

2024-01-25 14:52:55 -08:00

nft_counter.c

netfilter: nf_tables: memcg accounting for dynamically allocated objects

2024-01-25 14:52:54 -08:00

nft_ct.c

…

nft_dup_netdev.c

…

nft_dynset.c

…

nft_exthdr.c

…

nft_fib_inet.c

…

nft_fib_netdev.c

…

nft_fib.c

…

nft_flow_offload.c

netfilter: nf_tables: validate NFPROTO_* family

2024-02-23 08:54:29 +01:00

nft_fwd_netdev.c

…

nft_hash.c

…

nft_immediate.c

…

nft_last.c

netfilter: nft_last: copy content when cloning expression

2024-01-25 14:52:56 -08:00

nft_limit.c

netfilter: nft_limit: reject configurations that cause integer overflow

2024-02-23 08:54:29 +01:00

nft_log.c

…

nft_lookup.c

…

nft_masq.c

…

nft_meta.c

…

nft_nat.c

netfilter: nf_tables: validate NFPROTO_* family

2024-02-23 08:54:29 +01:00

nft_numgen.c

…

nft_objref.c

…

nft_osf.c

…

nft_payload.c

…

nft_queue.c

…

nft_quota.c

netfilter: nft_quota: copy content when cloning expression

2024-01-25 14:52:56 -08:00

nft_range.c

…

nft_redir.c

…

nft_reject_inet.c

…

nft_reject_netdev.c

…

nft_reject.c

…

nft_rt.c

netfilter: nf_tables: validate NFPROTO_* family

2024-02-23 08:54:29 +01:00

nft_set_bitmap.c

…

nft_set_hash.c

…

nft_set_pipapo_avx2.c

…

nft_set_pipapo_avx2.h

…

nft_set_pipapo.c

…

nft_set_pipapo.h

…

nft_set_rbtree.c

…

nft_socket.c

netfilter: nf_tables: validate NFPROTO_* family

2024-02-23 08:54:29 +01:00

nft_synproxy.c

netfilter: nf_tables: validate NFPROTO_* family

2024-02-23 08:54:29 +01:00

nft_tproxy.c

netfilter: nf_tables: validate NFPROTO_* family

2024-02-23 08:54:29 +01:00

nft_tunnel.c

…

nft_xfrm.c

netfilter: nf_tables: validate NFPROTO_* family

2024-02-23 08:54:29 +01:00

utils.c

…

x_tables.c

…

xt_addrtype.c

…

xt_AUDIT.c

…

xt_bpf.c

…

xt_cgroup.c

…

xt_CHECKSUM.c

…

xt_CLASSIFY.c

…

xt_cluster.c

…

xt_comment.c

…

xt_connbytes.c

…

xt_connlabel.c

…

xt_connlimit.c

…

xt_connmark.c

…

xt_CONNSECMARK.c

…

xt_conntrack.c

…

xt_cpu.c

…

xt_CT.c

…

xt_dccp.c

…

xt_devgroup.c

…

xt_dscp.c

…

xt_DSCP.c

…

xt_ecn.c

…

xt_esp.c

…

xt_hashlimit.c

…

xt_helper.c

…

xt_hl.c

…

xt_HL.c

…

xt_HMARK.c

…

xt_IDLETIMER.c

…

xt_ipcomp.c

…

xt_iprange.c

…

xt_ipvs.c

…

xt_l2tp.c

…

xt_LED.c

…

xt_length.c

…

xt_limit.c

…

xt_LOG.c

…

xt_mac.c

…

xt_mark.c

…

xt_MASQUERADE.c

…

xt_multiport.c

…

xt_nat.c

…

xt_NETMAP.c

…

xt_nfacct.c

…

xt_NFLOG.c

…

xt_NFQUEUE.c

…

xt_osf.c

…

xt_owner.c

…

xt_physdev.c

…

xt_pkttype.c

…

xt_policy.c

…

xt_quota.c

…

xt_rateest.c

…

xt_RATEEST.c

…

xt_realm.c

…

xt_recent.c

…

xt_REDIRECT.c

…

xt_repldata.h

…

xt_sctp.c

…

xt_SECMARK.c

…

xt_set.c

…

xt_socket.c

…

xt_state.c

…

xt_statistic.c

…

xt_string.c

…

xt_tcpmss.c

…

xt_TCPMSS.c

…

xt_TCPOPTSTRIP.c

…

xt_tcpudp.c

…

xt_TEE.c

…

xt_time.c

…

xt_TPROXY.c

…

xt_TRACE.c

…

xt_u32.c

…