Commit f0a98ae8 authored Dec 05, 2011 by Dan Carpenter Committed by David S. Miller Dec 06, 2011

openvswitch: small potential memory leak in ovs_vport_alloc()



We're unlikely to hit this leak, but the static checkers complain if we
don't take care of it.

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Acked-by: Jesse Gross <jesse@nicira.com>
Signed-off-by: David S. Miller <davem@davemloft.net>

parent 40e4783e

net/openvswitch/vport.c

+3 −1

Original line number	Diff line number	Diff line
		@@ -127,8 +127,10 @@ struct vport ovs_vport_alloc(int priv_size, const struct vport_ops ops,
		vport->ops = ops;

		vport->percpu_stats = alloc_percpu(struct vport_percpu_stats);
		if (!vport->percpu_stats)
		if (!vport->percpu_stats) {
		kfree(vport);
		return ERR_PTR(-ENOMEM);
		}

		spin_lock_init(&vport->stats_lock);

Admin message