diff options
| author | Hangyu Hua <hbh25y@gmail.com> | 2023-12-21 10:25:31 +0800 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2024-01-01 13:08:15 +0000 |
| commit | 8fcb0382af6f1ef50936f1be05b8149eb2f88496 (patch) | |
| tree | cd265670a21784980f490f518b98b98256564c8c /net/sched | |
| parent | dcea1bd45e6d111cc8fc1aaefa7e31694089bda3 (diff) | |
net: sched: em_text: fix possible memory leak in em_text_destroy()
m->data needs to be freed when em_text_destroy is called.
Fixes: d675c989ed2d ("[PKT_SCHED]: Packet classification based on textsearch (ematch)")
Acked-by: Jamal Hadi Salim <jhs@mojatatu.com>
Signed-off-by: Hangyu Hua <hbh25y@gmail.com>
Reviewed-by: Simon Horman <horms@kernel.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/sched')
| -rw-r--r-- | net/sched/em_text.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/net/sched/em_text.c b/net/sched/em_text.c index 6f3c1fb2fb44..f176afb70559 100644 --- a/net/sched/em_text.c +++ b/net/sched/em_text.c @@ -97,8 +97,10 @@ retry: static void em_text_destroy(struct tcf_ematch *m) { - if (EM_TEXT_PRIV(m) && EM_TEXT_PRIV(m)->config) + if (EM_TEXT_PRIV(m) && EM_TEXT_PRIV(m)->config) { textsearch_destroy(EM_TEXT_PRIV(m)->config); + kfree(EM_TEXT_PRIV(m)); + } } static int em_text_dump(struct sk_buff *skb, struct tcf_ematch *m) |