From 43ef49e75fbdac3f3067c87d36ce0b42637f96db Mon Sep 17 00:00:00 2001 From: Hung Nguyen Date: Wed, 19 Mar 2025 23:48:56 +0000 Subject: [PATCH] Use strings.Fields for whitespace splitting to fix issues with strings.Split in case of multiple consecutive spaces. --- pkg/gce-pd-csi-driver/cache.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/pkg/gce-pd-csi-driver/cache.go b/pkg/gce-pd-csi-driver/cache.go index 03b5dfe0f..1c5f4d192 100644 --- a/pkg/gce-pd-csi-driver/cache.go +++ b/pkg/gce-pd-csi-driver/cache.go @@ -45,7 +45,7 @@ func fetchRAIDedLocalSsdPath() (string, error) { return "", fmt.Errorf("Error getting RAIDed device path for Data Cache %v, output:%v", err, string(info)) } infoString := strings.TrimSpace(string(info)) - infoSlice := strings.Split(infoString, " ") + infoSlice := strings.Fields(infoString) // We want to get the second element in the array (sample: ARRAY /dev/md126 metadata=1.2 name=csi-driver-data-cache UUID=*), // which is the path to the RAIDed device @@ -357,7 +357,7 @@ func FetchAllLssds() ([]string, error) { for _, ssd := range infoList { ssd = strings.TrimSpace(ssd) if strings.HasPrefix(ssd, "/dev/nvme") { - ssdDetails := strings.Split(ssd, " ") + ssdDetails := strings.Fields(ssd) lssd := re.MatchString(ssdDetails[1]) if lssd { diskList = append(diskList, strings.TrimSpace(ssdDetails[0]))