]> git.ipfire.org Git - thirdparty/mdadm.git/commitdiff
Use dev_open in validate geometry container
authorBlazej Kucman <blazej.kucman@intel.com>
Tue, 15 Jun 2021 14:45:39 +0000 (16:45 +0200)
committerJes Sorensen <jsorensen@fb.com>
Fri, 16 Jul 2021 14:23:05 +0000 (10:23 -0400)
Fix regression caused by the patch 1f5d54a06
("Manage: Call validate_geometry when adding drive to external container")
- mdmonitor passes to Manage() routine dev name as min:mjr.
The open() used in validate_geometry_container()
in both ddf and imsm requires path, replace open calls by dev_open,
which allows to use dev path and min:mjr.

Signed-off-by: Blazej Kucman <blazej.kucman@intel.com>
Signed-off-by: Jes Sorensen <jsorensen@fb.com>
super-ddf.c
super-intel.c

index 80a40f84ab2f31a4dba7db40fb1d04b466fdacbf..dc8e512f36d4d247c4495c00ccfdbc32cda23424 100644 (file)
@@ -3463,7 +3463,7 @@ validate_geometry_ddf_container(struct supertype *st,
        if (!dev)
                return 1;
 
-       fd = open(dev, O_RDONLY|O_EXCL, 0);
+       fd = dev_open(dev, O_RDONLY|O_EXCL);
        if (fd < 0) {
                if (verbose)
                        pr_err("ddf: Cannot open %s: %s\n",
index fe45d93301b2bed50f85224b9edbff2d114f4d32..5356ca5144da36f57100bbcd1d513e9889bd88e3 100644 (file)
@@ -6668,7 +6668,7 @@ static int validate_geometry_imsm_container(struct supertype *st, int level,
        if (!dev)
                return 1;
 
-       fd = open(dev, O_RDONLY|O_EXCL, 0);
+       fd = dev_open(dev, O_RDONLY|O_EXCL);
        if (fd < 0) {
                if (verbose > 0)
                        pr_err("imsm: Cannot open %s: %s\n",