[src/trunk]: src/sys/arch/xen/xen Feeding uninitialised garbage to the hyperv...

[jnemeth <jnemeth%NetBSD.org@localhost>]

details:   https://anonhg.NetBSD.org/src/rev/061f7cb37974
branches:  trunk
changeset: 345774:061f7cb37974
user:      jnemeth <jnemeth%NetBSD.org@localhost>
date:      Wed Jun 08 01:59:06 2016 +0000

description:
Feeding uninitialised garbage to the hypervisor is likely to be a bad idea.

diffstat:

 sys/arch/xen/xen/xen_machdep.c |  5 +++--
 1 files changed, 3 insertions(+), 2 deletions(-)

diffs (26 lines):

diff -r 39f0d47197a1 -r 061f7cb37974 sys/arch/xen/xen/xen_machdep.c
--- a/sys/arch/xen/xen/xen_machdep.c    Wed Jun 08 01:41:43 2016 +0000
+++ b/sys/arch/xen/xen/xen_machdep.c    Wed Jun 08 01:59:06 2016 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: xen_machdep.c,v 1.14 2016/05/29 17:06:17 bouyer Exp $  */
+/*     $NetBSD: xen_machdep.c,v 1.15 2016/06/08 01:59:06 jnemeth Exp $ */
 
 /*
  * Copyright (c) 2006 Manuel Bouyer.
@@ -53,7 +53,7 @@
 
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: xen_machdep.c,v 1.14 2016/05/29 17:06:17 bouyer Exp $");
+__KERNEL_RCSID(0, "$NetBSD: xen_machdep.c,v 1.15 2016/06/08 01:59:06 jnemeth Exp $");
 
 #include "opt_xen.h"
 
@@ -406,6 +406,7 @@
        xen_feature_info_t features;
 
        for (int sm = 0; sm < XENFEAT_NR_SUBMAPS; sm++) {
+               features.submap_idx = sm;
                if (HYPERVISOR_xen_version(XENVER_get_features, &features) < 0)
                        break;
                for (int f = 0; f < 32; f++) {