From 102a6f0bed0c012f3484a44a6a0f5098fca1dab1 Mon Sep 17 00:00:00 2001
From: John Howard <jhoward@microsoft.com>
Date: Fri, 9 Nov 2018 14:38:53 -0800
Subject: [PATCH] Fix panic

Signed-off-by: John Howard <jhoward@microsoft.com>
---
 cmd/runhcs/container.go | 28 ++++++++++++++--------------
 1 file changed, 14 insertions(+), 14 deletions(-)

diff --git a/cmd/runhcs/container.go b/cmd/runhcs/container.go
index e42346abef..bb600157e0 100644
--- a/cmd/runhcs/container.go
+++ b/cmd/runhcs/container.go
@@ -390,53 +390,53 @@ func createContainer(cfg *containerConfig) (_ *container, err error) {
 
 		// Cater for fields that can be configured via OCI annotations
 		if v, ok := cfg.Spec.Annotations[annotationAllowOverCommit]; ok {
-			var allowOvercommit *bool
+			yes := true
+			no := false
 			switch strings.ToLower(v) {
 			case "true":
-				*allowOvercommit = true
+				opts.AllowOvercommit = &yes
 			case "false":
-				*allowOvercommit = false
+				opts.AllowOvercommit = &no
 			default:
 				return nil, fmt.Errorf("annotation %s must be true or false", annotationAllowOverCommit)
 			}
-			opts.AllowOvercommit = allowOvercommit
 		}
 
 		if v, ok := cfg.Spec.Annotations[annotationEnableDeferredCommit]; ok {
-			var enableDeferredCommit *bool
+			yes := true
+			no := false
 			switch strings.ToLower(v) {
 			case "true":
-				*enableDeferredCommit = true
+				opts.EnableDeferredCommit = &yes
 			case "false":
-				*enableDeferredCommit = false
+				opts.EnableDeferredCommit = &no
 			default:
 				return nil, fmt.Errorf("annotation %s must be true or false", annotationEnableDeferredCommit)
 			}
-			opts.EnableDeferredCommit = enableDeferredCommit
 		}
 
 		if v, ok := cfg.Spec.Annotations[annotationVPMemCount]; ok {
 			var (
 				countu uint64
-				counti *uint32
+				counti uint32
 				err    error
 			)
 			if countu, err = strconv.ParseUint(v, 10, 32); err != nil {
 				return nil, fmt.Errorf("annotation %s could not be parsed: %s", annotationVPMemCount, err)
 			}
-			*counti = uint32(countu)
-			opts.VPMemDeviceCount = counti
+			counti = uint32(countu)
+			opts.VPMemDeviceCount = &counti
 		}
 
 		if v, ok := cfg.Spec.Annotations[annotationVPMemSize]; ok {
 			var (
-				countu *uint64
+				countu uint64
 				err    error
 			)
-			if *countu, err = strconv.ParseUint(v, 10, 64); err != nil {
+			if countu, err = strconv.ParseUint(v, 10, 64); err != nil {
 				return nil, fmt.Errorf("annotation %s could not be parsed: %s", annotationVPMemSize, err)
 			}
-			opts.VPMemSizeBytes = countu
+			opts.VPMemSizeBytes = &countu
 		}
 
 		shim, err := c.startVMShim(cfg.VMLogFile, opts)