Add ANNOUNCE-2.3.1

[thirdparty/mdadm.git] / Examine.c

diff --git a/Examine.c b/Examine.c
index 33fa1e5fccc7d70b50712cd94ffcc8b9bd8abc68..fcfb02b8d006a1d707758f6247f84e4201b4c0a3 100644 (file)
--- a/Examine.c
+++ b/Examine.c
@@ -51,12 +51,12 @@ int Examine(mddev_dev_t devlist, int brief, int scan, int SparcAdjust, struct su
         *   utime, state etc
         *
         * If (brief) gather devices for same array and just print a mdadm.conf line including devices=
-        * if devlist==NULL, use conf_get_devs(
+        * if devlist==NULL, use conf_get_devs()
         */
        int fd; 
        void *super = NULL;
        int rv = 0;
-       int err;
+       int err = 0;
 
        struct array {
                void *super;
@@ -71,7 +71,7 @@ int Examine(mddev_dev_t devlist, int brief, int scan, int SparcAdjust, struct su
        for (; devlist ; devlist=devlist->next) {
                struct supertype *st = forcest;
 
-               fd = open(devlist->devname, O_RDONLY);
+               fd = dev_open(devlist->devname, O_RDONLY);
                if (fd < 0) {
                        if (!scan)
                                fprintf(stderr,Name ": cannot open %s: %s\n",
@@ -83,13 +83,17 @@ int Examine(mddev_dev_t devlist, int brief, int scan, int SparcAdjust, struct su
                                st = guess_super(fd);
                        if (st)
                                err = st->ss->load_super(st, fd, &super, (brief||scan)?NULL:devlist->devname);
-                       else
+                       else {
+                               if (!brief)
+                                       fprintf(stderr, Name ": No md superblock detected on %s.\n", devlist->devname);
                                err = 1;
+                       }
                        close(fd);
                }
-               if (err)
+               if (err) {
+                       rv = 1;
                        continue;
-               if (err) rv =1;
+               }
 
                if (SparcAdjust)
                        st->ss->update_super(NULL, super, "sparc2.2", devlist->devname,  0);
@@ -140,7 +144,8 @@ int Examine(mddev_dev_t devlist, int brief, int scan, int SparcAdjust, struct su
                        }
                        free(ap->super);
                        /* FIXME free ap */
-                       printf("\n");
+                       if (ap->spares || brief > 1)
+                               printf("\n");
                }
        }
        return rv;