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