[PATCH V1] Fix for Coverity ID: 1461759

Alexandru Isaila posted 1 patch 4 years ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/xen tags/patchew/20200415135313.12886-1-aisaila@bitdefender.com
xen/arch/x86/hvm/hvm.c | 1 +
1 file changed, 1 insertion(+)
[PATCH V1] Fix for Coverity ID: 1461759
Posted by Alexandru Isaila 4 years ago
Signed-off-by: Alexandru Isaila <aisaila@bitdefender.com>

---
CC: Jan Beulich <jbeulich@suse.com>
CC: Andrew Cooper <andrew.cooper3@citrix.com>
CC: Wei Liu <wl@xen.org>
CC: "Roger Pau Monné" <roger.pau@citrix.com>
---
 xen/arch/x86/hvm/hvm.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/xen/arch/x86/hvm/hvm.c b/xen/arch/x86/hvm/hvm.c
index 6f6f3f73a8..45959d3412 100644
--- a/xen/arch/x86/hvm/hvm.c
+++ b/xen/arch/x86/hvm/hvm.c
@@ -4798,6 +4798,7 @@ static int do_altp2m_op(
         else
             rc = p2m_set_altp2m_view_visibility(d, idx,
                                                 a.u.set_visibility.visible);
+        break;
     }
 
     default:
-- 
2.17.1


Re: [PATCH V1] Fix for Coverity ID: 1461759
Posted by Wei Liu 4 years ago
On Wed, Apr 15, 2020 at 04:53:13PM +0300, Alexandru Isaila wrote:
> Signed-off-by: Alexandru Isaila <aisaila@bitdefender.com>

Thanks, but Roger already posted a fix for this.

Wei.

Re: [PATCH V1] Fix for Coverity ID: 1461759
Posted by Alexandru Stefan ISAILA 4 years ago
________________________________
From: Xen-devel <xen-devel-bounces@lists.xenproject.org> on behalf of Wei Liu <wl@xen.org>
Sent: Wednesday, April 15, 2020 4:59 PM
To: Alexandru Stefan ISAILA <aisaila@bitdefender.com>
Cc: xen-devel@lists.xenproject.org <xen-devel@lists.xenproject.org>; Roger Pau Monné <roger.pau@citrix.com>; Wei Liu <wl@xen.org>; Jan Beulich <jbeulich@suse.com>; Andrew Cooper <andrew.cooper3@citrix.com>
Subject: Re: [PATCH V1] Fix for Coverity ID: 1461759

On Wed, Apr 15, 2020 at 04:53:13PM +0300, Alexandru Isaila wrote:
> Signed-off-by: Alexandru Isaila <aisaila@bitdefender.com>

Thanks, but Roger already posted a fix for this.

Sorry about the late reaction to this, I did no check my email.

Alex