overlay: do not append empty option

if d.options.mountOptions is the empty string, we should not append an empty option. Signed-off-by: Giuseppe Scrivano <[email protected]>
containers · Sep 13, 2023 · f109925 · f109925
1 parent 0c47206
commit f109925
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/drivers/overlay/overlay.go b/drivers/overlay/overlay.go
@@ -1447,7 +1447,9 @@ func (d *Driver) get(id string, disableShifting bool, options graphdriver.MountO
 	needsIDMapping := !disableShifting && len(options.UidMaps) > 0 && len(options.GidMaps) > 0 && d.options.mountProgram == ""
 
 	if len(optsList) == 0 {
-		optsList = strings.Split(d.options.mountOptions, ",")
+		if d.options.mountOptions != "" {
+			optsList = strings.Split(d.options.mountOptions, ",")
+		}
 	} else {
 		// If metacopy=on is present in d.options.mountOptions it must be present in the mount
 		// options otherwise the kernel refuses to follow the metacopy xattr.