2b8eb379930 changed the type of i to be unsigned, but the inner loop
depends on it being a signed type.

Coverity-ID: 1413017

Signed-off-by: Wei Liu <wei.l...@citrix.com>
---
Cc: Tim Deegan <t...@xen.org>
Cc: George Dunlap <george.dun...@eu.citrix.com>
Cc: Jan Beulich <jbeul...@suse.com>
Cc: Andrew Cooper <andrew.coop...@citrix.com>
---
 xen/arch/x86/mm.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/xen/arch/x86/mm.c b/xen/arch/x86/mm.c
index b20f37f253..b8c705854a 100644
--- a/xen/arch/x86/mm.c
+++ b/xen/arch/x86/mm.c
@@ -2681,7 +2681,7 @@ int get_page_type_preemptible(struct page_info *page, 
unsigned long type)
 
 static int get_spage_pages(struct page_info *page, struct domain *d)
 {
-    unsigned int i;
+    int i;
 
     for ( i = 0; i < (1u << PAGETABLE_ORDER); i++, page++ )
     {
-- 
2.11.0


_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel

Reply via email to