]> xenbits.xensource.com Git - xen.git/commitdiff
x86/shadow: correct an inverted conditional in dirty VRAM tracking
authorJan Beulich <jbeulich@suse.com>
Tue, 7 Jul 2020 13:09:25 +0000 (15:09 +0200)
committerJan Beulich <jbeulich@suse.com>
Tue, 7 Jul 2020 13:09:25 +0000 (15:09 +0200)
This originally was "mfn_x(mfn) == INVALID_MFN". Make it like this
again, taking the opportunity to also drop the unnecessary nearby
braces.

This is XSA-319.

Fixes: 246a5a3377c2 ("xen: Use a typesafe to define INVALID_MFN")
Signed-off-by: Jan Beulich <jbeulich@suse.com>
Reviewed-by: Andrew Cooper <andrew.cooper3@citrix.com>
master commit: 23a216f99d40fbfbc2318ade89d8213eea6ba1f8
master date: 2020-07-07 14:36:24 +0200

xen/arch/x86/mm/shadow/common.c

index 1ac5b22028dfc4da006aeb9b77c6b0d9329333ff..6595fba04c5d53e987bdd00619e7749135674412 100644 (file)
@@ -3373,10 +3373,8 @@ int shadow_track_dirty_vram(struct domain *d,
             int dirty = 0;
             paddr_t sl1ma = dirty_vram->sl1ma[i];
 
-            if ( !mfn_eq(mfn, INVALID_MFN) )
-            {
+            if ( mfn_eq(mfn, INVALID_MFN) )
                 dirty = 1;
-            }
             else
             {
                 page = mfn_to_page(mfn);