Messages in this thread | | | From | KY Srinivasan <> | Subject | RE: [PATCH 2/2] tools: hv: fcopy: set 'error' in case an unknown operation was requested | Date | Mon, 17 Sep 2018 14:16:48 +0000 |
| |
> -----Original Message----- > From: Greg KH <gregkh@linuxfoundation.org> > Sent: Sunday, September 16, 2018 9:56 PM > To: KY Srinivasan <kys@microsoft.com> > Cc: linux-kernel@vger.kernel.org; devel@linuxdriverproject.org; > olaf@aepfle.de; apw@canonical.com; jasowang@redhat.com; Stephen > Hemminger <sthemmin@microsoft.com>; Michael Kelley (EOSG) > <Michael.H.Kelley@microsoft.com>; vkuznets <vkuznets@redhat.com> > Subject: Re: [PATCH 2/2] tools: hv: fcopy: set 'error' in case an unknown > operation was requested > > On Mon, Sep 17, 2018 at 04:14:55AM +0000, kys@linuxonhyperv.com wrote: > > From: Vitaly Kuznetsov <vkuznets@redhat.com> > > > > 'error' variable is left uninitialized in case we see an unknown operation. > > As we don't immediately return and proceed to pwrite() we need to set it > > to something, HV_E_FAIL sounds good enough. > > > > Signed-off-by: Vitaly Kuznetsov <vkuznets@redhat.com> > > Signed-off-by: K. Y. Srinivasan <kys@microsoft.com> > > --- > > tools/hv/hv_fcopy_daemon.c | 1 + > > 1 file changed, 1 insertion(+) > > No need to backport for stable? Thanks for pointing this out. Should I resubmit with the stable tag?
Thanks,
K. Y
| |