lkml.org 
[lkml]   [2022]   [Aug]   [19]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 5.10 073/545] netfilter: nf_tables: do not allow CHAIN_ID to refer to another table
    Date
    From: Thadeu Lima de Souza Cascardo <cascardo@canonical.com>

    commit 95f466d22364a33d183509629d0879885b4f547e upstream.

    When doing lookups for chains on the same batch by using its ID, a chain
    from a different table can be used. If a rule is added to a table but
    refers to a chain in a different table, it will be linked to the chain in
    table2, but would have expressions referring to objects in table1.

    Then, when table1 is removed, the rule will not be removed as its linked to
    a chain in table2. When expressions in the rule are processed or removed,
    that will lead to a use-after-free.

    When looking for chains by ID, use the table that was used for the lookup
    by name, and only return chains belonging to that same table.

    Fixes: 837830a4b439 ("netfilter: nf_tables: add NFTA_RULE_CHAIN_ID attribute")
    Signed-off-by: Thadeu Lima de Souza Cascardo <cascardo@canonical.com>
    Cc: <stable@vger.kernel.org>
    Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
    Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
    ---
    net/netfilter/nf_tables_api.c | 6 ++++--
    1 file changed, 4 insertions(+), 2 deletions(-)

    --- a/net/netfilter/nf_tables_api.c
    +++ b/net/netfilter/nf_tables_api.c
    @@ -2265,6 +2265,7 @@ err:
    }

    static struct nft_chain *nft_chain_lookup_byid(const struct net *net,
    + const struct nft_table *table,
    const struct nlattr *nla)
    {
    u32 id = ntohl(nla_get_be32(nla));
    @@ -2274,6 +2275,7 @@ static struct nft_chain *nft_chain_looku
    struct nft_chain *chain = trans->ctx.chain;

    if (trans->msg_type == NFT_MSG_NEWCHAIN &&
    + chain->table == table &&
    id == nft_trans_chain_id(trans))
    return chain;
    }
    @@ -3199,7 +3201,7 @@ static int nf_tables_newrule(struct net
    return -EOPNOTSUPP;

    } else if (nla[NFTA_RULE_CHAIN_ID]) {
    - chain = nft_chain_lookup_byid(net, nla[NFTA_RULE_CHAIN_ID]);
    + chain = nft_chain_lookup_byid(net, table, nla[NFTA_RULE_CHAIN_ID]);
    if (IS_ERR(chain)) {
    NL_SET_BAD_ATTR(extack, nla[NFTA_RULE_CHAIN_ID]);
    return PTR_ERR(chain);
    @@ -8671,7 +8673,7 @@ static int nft_verdict_init(const struct
    tb[NFTA_VERDICT_CHAIN],
    genmask);
    } else if (tb[NFTA_VERDICT_CHAIN_ID]) {
    - chain = nft_chain_lookup_byid(ctx->net,
    + chain = nft_chain_lookup_byid(ctx->net, ctx->table,
    tb[NFTA_VERDICT_CHAIN_ID]);
    if (IS_ERR(chain))
    return PTR_ERR(chain);

    \
     
     \ /
      Last update: 2022-08-19 17:55    [W:4.170 / U:0.152 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site