Revert UnmarshalMetadata returning pointer

cmd/sops/common/common.go

@@ -78,7 +78,7 @@ func LoadEncryptedFile(inputStore sops.Store, inputPath string) (*sops.Tree, err
 	}
 	tree := sops.Tree{
 		Branch:   branch,
-		Metadata: *metadata,
+		Metadata: metadata,
 	}
 	return &tree, nil
 }

cmd/sops/edit.go

@@ -214,7 +214,7 @@ func runEditorUntilOk(opts runEditorUntilOkOpts) error {
 				bufio.NewReader(os.Stdin).ReadByte()
 				continue
 			}
-			opts.Tree.Metadata = *metadata
+			opts.Tree.Metadata = metadata
 		}
 		opts.Tree.Branch = newBranch
 		needVersionUpdated, err := AIsNewerThanB(version, opts.Tree.Metadata.Version)

decrypt/decrypt.go

@@ -53,7 +53,7 @@ func Data(data []byte, format string) (cleartext []byte, err error) {
 	if err != nil {
 		return nil, err
 	}
-	tree := sops.Tree{Branch: branch, Metadata: *metadata}
+	tree := sops.Tree{Branch: branch, Metadata: metadata}
 
 	// Decrypt the tree
 	cipher := aes.Cipher{}

sops.go

@@ -311,7 +311,7 @@ type KeyGroup []keys.MasterKey
 // Store provides a way to load and save the sops tree along with metadata
 type Store interface {
 	Unmarshal(in []byte) (TreeBranch, error)
-	UnmarshalMetadata(in []byte) (*Metadata, error)
+	UnmarshalMetadata(in []byte) (Metadata, error)
 	Marshal(TreeBranch) ([]byte, error)
 	MarshalWithMetadata(TreeBranch, Metadata) ([]byte, error)
 	MarshalValue(interface{}) ([]byte, error)

stores/json/store.go

@@ -54,7 +54,7 @@ func (store BinaryStore) Unmarshal(in []byte) (sops.TreeBranch, error) {
 }
 
 // UnmarshalMetadata takes a binary format sops file and extracts sops' metadata from it
-func (store BinaryStore) UnmarshalMetadata(in []byte) (*sops.Metadata, error) {
+func (store BinaryStore) UnmarshalMetadata(in []byte) (sops.Metadata, error) {
 	return store.store.UnmarshalMetadata(in)
 }
 
@@ -248,11 +248,11 @@ func (store Store) MarshalValue(v interface{}) ([]byte, error) {
 }
 
 // UnmarshalMetadata takes a json string and extracts sops' metadata from it
-func (store Store) UnmarshalMetadata(in []byte) (*sops.Metadata, error) {
+func (store Store) UnmarshalMetadata(in []byte) (sops.Metadata, error) {
 	file := stores.SopsFile{}
 	err := json.Unmarshal(in, &file)
 	if err != nil {
-		return nil, fmt.Errorf("Error unmarshalling input json: %s", err)
+		return sops.Metadata{}, fmt.Errorf("Error unmarshalling input json: %s", err)
 	}
 	return file.Metadata.ToInternal()
 }

stores/stores.go

@@ -101,19 +101,19 @@ func kmsKeysFromGroup(group sops.KeyGroup) (keys []kmskey) {
 	return
 }
 
-func (m *Metadata) ToInternal() (*sops.Metadata, error) {
+func (m *Metadata) ToInternal() (sops.Metadata, error) {
 	lastModified, err := time.Parse(time.RFC3339, m.LastModified)
 	if err != nil {
-		return nil, err
+		return sops.Metadata{}, err
 	}
 	groups, err := m.internalKeygroups()
 	if err != nil {
-		return nil, err
+		return sops.Metadata{}, err
 	}
 	if m.UnencryptedSuffix == "" {
 		m.UnencryptedSuffix = sops.DefaultUnencryptedSuffix
 	}
-	return &sops.Metadata{
+	return sops.Metadata{
 		KeyGroups:                 groups,
 		ShamirQuorum:              m.ShamirQuorum,
 		Version:                   m.Version,

stores/yaml/store.go

@@ -141,11 +141,11 @@ func (store Store) MarshalValue(v interface{}) ([]byte, error) {
 }
 
 // UnmarshalMetadata takes a yaml document as a string and extracts sops' metadata from it
-func (s *Store) UnmarshalMetadata(in []byte) (*sops.Metadata, error) {
+func (s *Store) UnmarshalMetadata(in []byte) (sops.Metadata, error) {
 	file := stores.SopsFile{}
 	err := yaml.Unmarshal(in, &file)
 	if err != nil {
-		return nil, fmt.Errorf("Error unmarshalling input yaml: %s", err)
+		return sops.Metadata{}, fmt.Errorf("Error unmarshalling input yaml: %s", err)
 	}
 	return file.Metadata.ToInternal()
 }