On Tue, Dec 13, 2016 at 05:34:03PM +0100, Richard Cochran wrote:
> On Tue, Dec 13, 2016 at 04:47:23PM +0100, Miroslav Lichvar wrote:
> > > + if (commandline) {
> > > +         dst->flags &= CFG_ITEM_LOCKED;
> > 
> > Should this be dst->flags |= CFG_ITEM_LOCKED ?
> 
> Ouch.  Yes of course.

Ok, with this change, valgrind reports no errors and the options seem
to be working as expected. This is a nice feature! Could you please
mention it in the man page?

Thanks,

-- 
Miroslav Lichvar

------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most 
engaging tech sites, SlashDot.org! http://sdm.link/slashdot
_______________________________________________
Linuxptp-devel mailing list
Linuxptp-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linuxptp-devel

Reply via email to