summaryrefslogtreecommitdiff
path: root/lib
diff options
context:
space:
mode:
authorHarry van Haaren <harry.van.haaren@intel.com>2021-08-12 15:57:34 +0000
committerIan Stokes <ian.stokes@intel.com>2021-08-16 15:52:55 +0100
commit01cbe1ed4135175cb1da97587599889eb60fc88b (patch)
treeff4183fb08052f306381e16e5c276e8330bd981c /lib
parentd2ad305a6438c69121fd012f417b2779cc0cadbe (diff)
downloadopenvswitch-01cbe1ed4135175cb1da97587599889eb60fc88b.tar.gz
dpif-netdev: fix memory leak in dpcls subtable set command
This patch fixes a memory leak when the command "dpif-netdev/subtable-lookup-prio-set" is run, the pmd_list required to iterate the PMD threads was not being freed. This issue was identified by a static analysis tool. Fixes: 3d018c3e ("dpif-netdev: Add subtable lookup prio set command.") Signed-off-by: Harry van Haaren <harry.van.haaren@intel.com> Acked-by: Kevin Traynor <ktraynor@redhat.com> Signed-off-by: Ian Stokes <ian.stokes@intel.com>
Diffstat (limited to 'lib')
-rw-r--r--lib/dpif-netdev.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/lib/dpif-netdev.c b/lib/dpif-netdev.c
index d88c5cba7..80531448a 100644
--- a/lib/dpif-netdev.c
+++ b/lib/dpif-netdev.c
@@ -994,6 +994,7 @@ dpif_netdev_subtable_lookup_set(struct unixctl_conn *conn, int argc OVS_UNUSED,
/* release port mutex before netdev mutex. */
ovs_mutex_unlock(&dp->port_mutex);
+ free(pmd_list);
}
ovs_mutex_unlock(&dp_netdev_mutex);