On Mon, Mar 07, 2016 at 01:00:28PM -0800, Justin Pettit wrote: > > > On Mar 7, 2016, at 11:19 AM, Ben Pfaff <b...@ovn.org> wrote: > > > > + if (VLOG_IS_DBG_ENABLED()) { > > + char *params = json_to_string(request->params, 0); > > + char *id = json_to_string(request->id, 0); > > + VLOG_DBG("received request %s%s, id=%s", request->method, params, > > id); > > + free(params); > > + free(id); > > + } > > Not a big deal, but the containing function has a variable called "params" of > a different type.
Thank you for pointing that out. I added an _s suffix to params and id here. > Acked-by: Justin Pettit <jpet...@ovn.org> Thanks, I applied this to master. _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev