NPE

Signed-off-by: Daniel Valdivia <[email protected]>

Author: dvaldivia

restapi/user_buckets.go

@@ -308,20 +308,16 @@ func getAccountInfo(ctx context.Context, client MinioAdmin) ([]*models.Bucket, e
 	var bucketInfos []*models.Bucket
 	for _, bucket := range info.Buckets {
 
-		var tags map[string]string
+		tags := make(map[string]string)
 		if bucket.Details != nil && bucket.Details.Tagging != nil {
 			tags = bucket.Details.Tagging.ToMap()
 		}
 
 		bucketElem := &models.Bucket{
 			CreationDate: bucket.Created.Format(time.RFC3339),
 			Details: &models.BucketDetails{
-				Locking:             bucket.Details.Locking,
-				Quota:               nil,
-				Replication:         bucket.Details.Replication,
-				Tags:                tags,
-				Versioning:          bucket.Details.Versioning,
-				VersioningSuspended: bucket.Details.VersioningSuspended,
+				Quota: nil,
+				Tags:  tags,
 			},
 			RwAccess: &models.BucketRwAccess{
 				Read:  bucket.Access.Read,
@@ -331,12 +327,20 @@ func getAccountInfo(ctx context.Context, client MinioAdmin) ([]*models.Bucket, e
 			Objects: int64(bucket.Objects),
 			Size:    int64(bucket.Size),
 		}
-		if bucket.Details != nil && bucket.Details.Quota != nil {
-			bucketElem.Details.Quota = &models.BucketDetailsQuota{
-				Quota: int64(bucket.Details.Quota.Quota),
-				Type:  string(bucket.Details.Quota.Type),
+
+		if bucket.Details != nil {
+			bucketElem.Details.Locking = bucket.Details.Locking
+			bucketElem.Details.Replication = bucket.Details.Replication
+			bucketElem.Details.Versioning = bucket.Details.Versioning
+			bucketElem.Details.VersioningSuspended = bucket.Details.VersioningSuspended
+			if bucket.Details.Quota != nil {
+				bucketElem.Details.Quota = &models.BucketDetailsQuota{
+					Quota: int64(bucket.Details.Quota.Quota),
+					Type:  string(bucket.Details.Quota.Type),
+				}
 			}
 		}
+
 		bucketInfos = append(bucketInfos, bucketElem)
 	}
 	return bucketInfos, nil