[ovs-dev] [PATCH 1/8] ofproto: Add user-specifiable datapath description (OpenFlow 1.0)

Ben Pfaff blp at nicira.com
Fri Jan 22 09:17:16 PST 2010


On Fri, Jan 22, 2010 at 01:09:06AM -0800, Justin Pettit wrote:
> In OpenFlow 1.0, a "dp_desc" character array was added to the ofp_desc_stats
> structure that allows a human readable description of the datapath to be
> provided.

It looks OK relative to what's there now, but I think that we have a
memory leak since ofproto_destroy() does not free manufacturer,
hardware, software, serial, or dp_desc.  Am I right about that?  If so,
would you mind fixing that at the same time?




More information about the dev mailing list