virtcontainers: change pass by value to pass by reference

container.config does not point to sandbox.config.Containers.ContainerConfig
which caused the ContainerConfig not sync.

Fixes: #2129

Signed-off-by: Wang Liang <wangliangzz@inspur.com>

virtcontainers/container.go

@@ -717,7 +717,7 @@ func (c *Container) createBlockDevices() error {
 }
 
 // newContainer creates a Container structure from a sandbox and a container configuration.
-func newContainer(sandbox *Sandbox, contConfig ContainerConfig) (*Container, error) {
+func newContainer(sandbox *Sandbox, contConfig *ContainerConfig) (*Container, error) {
 	span, _ := sandbox.trace("newContainer")
 	defer span.Finish()
 
@@ -729,7 +729,7 @@ func newContainer(sandbox *Sandbox, contConfig ContainerConfig) (*Container, err
 		id:            contConfig.ID,
 		sandboxID:     sandbox.id,
 		rootFs:        contConfig.RootFs,
-		config:        &contConfig,
+		config:        contConfig,
 		sandbox:       sandbox,
 		runPath:       store.ContainerRuntimeRootPath(sandbox.id, contConfig.ID),
 		configPath:    store.ContainerConfigurationRootPath(sandbox.id, contConfig.ID),
@@ -785,7 +785,7 @@ func newContainer(sandbox *Sandbox, contConfig ContainerConfig) (*Container, err
 		return nil, err
 	}
 
-	if err = c.createDevices(contConfig); err != nil {
+	if err = c.createDevices(*contConfig); err != nil {
 		return nil, err
 	}
 

virtcontainers/sandbox.go

@@ -1099,7 +1099,7 @@ func (s *Sandbox) fetchContainers() error {
 		contConfig.Spec = &spec
 		s.config.Containers[i] = contConfig
 
-		c, err := newContainer(s, contConfig)
+		c, err := newContainer(s, &s.config.Containers[i])
 		if err != nil {
 			return err
 		}
@@ -1118,7 +1118,7 @@ func (s *Sandbox) fetchContainers() error {
 func (s *Sandbox) CreateContainer(contConfig ContainerConfig) (VCContainer, error) {
 	storeAlreadyExists := store.VCContainerStoreExists(s.ctx, s.id, contConfig.ID)
 	// Create the container.
-	c, err := newContainer(s, contConfig)
+	c, err := newContainer(s, &contConfig)
 	if err != nil {
 		return nil, err
 	}
@@ -1417,7 +1417,7 @@ func (s *Sandbox) createContainers() error {
 
 	for _, contConfig := range s.config.Containers {
 
-		c, err := newContainer(s, contConfig)
+		c, err := newContainer(s, &contConfig)
 		if err != nil {
 			return err
 		}