On Tue, 2017-05-30 at 08:35 -0700, Ben Pfaff wrote: > On Sat, May 27, 2017 at 06:26:00PM -0700, Greg Rose wrote: > > On Fri, 2017-05-26 at 16:07 -0700, Ben Pfaff wrote: > > > Found by Coverity. > > > > > > Reported-at: > > > https://scan3.coverity.com/reports.htm#v16889/p10449/fileInstanceId=14762909&defectInstanceId=4305311&mergedDefectId=180415 > > > Signed-off-by: Ben Pfaff <[email protected]> > > > --- > > > lib/dpctl.c | 3 ++- > > > 1 file changed, 2 insertions(+), 1 deletion(-) > > > > > > diff --git a/lib/dpctl.c b/lib/dpctl.c > > > index 11be85706db5..c0da3108943d 100644 > > > --- a/lib/dpctl.c > > > +++ b/lib/dpctl.c > > > @@ -1,5 +1,5 @@ > > > /* > > > - * Copyright (c) 2008, 2009, 2010, 2011, 2012, 2013, 2014, 2015, 2016 > > > Nicira, Inc. > > > + * Copyright (c) 2008, 2009, 2010, 2011, 2012, 2013, 2014, 2015, 2016, > > > 2017 Nicira, Inc. > > > > OK, last time I'll comment on this I swear! :) > > > > Better something like Copyright (c) 2008-2017 > > Did you look at the substantive part of the patch? > > Thanks, > > Ben.
Well yes, good catch by coverity. There's not much to it but it looks correct to me. Thanks, - Greg _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
