On Fri, Apr 12, 2013 at 02:51:37PM -0700, Gurucharan Shetty wrote:
> On Fri, Apr 12, 2013 at 10:03 AM, Ben Pfaff <[email protected]> wrote:
> 
> > On Wed, Apr 10, 2013 at 12:00:53PM -0700, Gurucharan Shetty wrote:
> > > The following two commands results in a ovs-vsctl segfault.
> > > ovs-vsctl -vfatal_signal:off --timeout=0 wait-until \
> > > Open_vswitch . external_ids:blah="1"
> > > /etc/init.d/openvswitch-switch restart
> > >
> > > This patch fixes the segfault by properly setting the global
> > > varibale, the_idl_txn to NULL when the underlying memory is
> > > freed.
> > >
> > > Signed-off-by: Gurucharan Shetty <[email protected]>
> >
> > Good catch.  This looks good to me.
> >
> > If this needs backporting, please do so.
> >
> Thanks. I applied this to master and 1.10.
> I don't think it is critical (other than segfault messages in dmesg) as
> restarting ovsdb-server also means that ovs-vsctl needs to reconnect. So it
> probably is not needed for 1.9 and 1.4.

I don't like segfaults.  If it's present in 1.9 then would you mind
fixing it there?  I doubt that this goes as far back as 1.4.
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev

Reply via email to