Do not use &internal.ObjAttr when you mean nil!

Seagate · Dec 21, 2024 · 730c271 · 730c271
1 parent 2d0e64a
commit 730c271
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/component/azstorage/block_blob.go b/component/azstorage/block_blob.go
@@ -569,7 +569,7 @@ func (bb *BlockBlob) List(prefix string, marker *string, count int32) ([]*intern
 	var dirList = make(map[string]bool)
 	for _, blobInfo := range listBlob.Segment.BlobItems {
 		blobInfo.Name = bb.getFileName(*blobInfo.Name)
-		attr := &internal.ObjAttr{}
+		var attr *internal.ObjAttr
 		if blobInfo.Properties.CustomerProvidedKeySHA256 != nil && *blobInfo.Properties.CustomerProvidedKeySHA256 != "" {
 			log.Trace("BlockBlob::List : blob is encrypted with customer provided key so fetching metadata explicitly using REST")
 			attr, err = bb.getAttrUsingRest(*blobInfo.Name)

diff --git a/component/loopback/loopback_fs.go b/component/loopback/loopback_fs.go
@@ -354,7 +354,7 @@ func (lfs *LoopbackFS) GetAttr(options internal.GetAttrOptions) (*internal.ObjAt
 	info, err := os.Lstat(path)
 	if err != nil {
 		log.Err("LoopbackFS::GetAttr : error [%s]", err)
-		return &internal.ObjAttr{}, err
+		return nil, err
 	}
 	attr := &internal.ObjAttr{
 		Path:  options.Name,