Use pointer receiver to load shim state.

Using value recerver reads information from the path, but it doesn't put the
information to the state variables.

PiperOrigin-RevId: 900431932

Author: milantracy

pkg/shim/v1/runsc/BUILD

@@ -63,13 +63,15 @@ go_test(
     srcs = [
         "oom_v2_test.go",
         "service_test.go",
+        "state_test.go",
     ],
     library = ":runsc",
     deps = [
         "//pkg/shim/v1/utils",
         "@com_github_containerd_cgroups//v2:go_default_library",
         "@com_github_containerd_containerd//events:go_default_library",
         "@com_github_containerd_containerd//runtime:go_default_library",
+        "@com_github_google_go_cmp//cmp:go_default_library",
         "@com_github_opencontainers_runtime_spec//specs-go:go_default_library",
     ],
 )

pkg/shim/v1/runsc/state.go

@@ -32,17 +32,17 @@ type State struct {
 }
 
 // Load loads the state from the given path.
-func (s State) Load(path string) error {
+func (s *State) Load(path string) error {
 	data, err := os.ReadFile(filepath.Join(path, filename))
 	if err != nil {
 		return err
 	}
-	return json.Unmarshal(data, &s)
+	return json.Unmarshal(data, s)
 }
 
 // Save saves the state to the given path.
-func (s State) Save(path string) error {
-	data, err := json.Marshal(&s)
+func (s *State) Save(path string) error {
+	data, err := json.Marshal(s)
 	if err != nil {
 		return err
 	}

pkg/shim/v1/runsc/state_test.go

@@ -0,0 +1,51 @@
+// Copyright 2026 The gVisor Authors.
+//
+// Licensed under the Apache License, Version 2.0 (the "License");
+// you may not use this file except in compliance with the License.
+// You may obtain a copy of the License at
+//
+//     https://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
+
+package runsc
+
+import (
+	"testing"
+
+	"github.com/google/go-cmp/cmp"
+)
+
+func TestState(t *testing.T) {
+	tmpdir := t.TempDir()
+	s := State{
+		Rootfs: "rootfs_path",
+		Options: Options{
+			ShimCgroup: "shim_cgroup",
+			IoUID:      123,
+			IoGID:      456,
+			BinaryName: "runsc",
+			Root:       "runsc_root",
+			LogLevel:   "info",
+			LogPath:    "logpath",
+			RunscConfig: map[string]string{
+				"flag1": "value1",
+				"flag2": "value2",
+			},
+		},
+	}
+	if err := s.Save(tmpdir); err != nil {
+		t.Fatalf("Save failed: %v", err)
+	}
+	var s2 State
+	if err := s2.Load(tmpdir); err != nil {
+		t.Fatalf("Load failed: %v", err)
+	}
+	if diff := cmp.Diff(s, s2); diff != "" {
+		t.Errorf("State is not equal, wanted:\n%v\ngot:\n%v\n", s, s2)
+	}
+}