From 7c3367585ec04e249dca1d6e64afcb2be9723a9b Mon Sep 17 00:00:00 2001 From: Lukasz Orlowski Date: Mon, 7 Nov 2011 12:20:34 +1100 Subject: [PATCH] fix: Allowed to assemble 2 volumes with the same names from config file. mdadm allowes to assemble 2 volumes with the same names based on the config file. The issue is fixed by iterating over the list of md device identifiers and comparing the names of md devices against each other, detecting identical names and blocking the assembly should the same names be found. Now having detected duplicate names, mdadm terminates without assembling the container, displaying appropriate prompt. Signed-off-by: Lukasz Orlowski Signed-off-by: NeilBrown --- config.c | 33 +++++++++++++++++++++++++++++++++ mdadm.c | 7 +++++++ mdadm.h | 1 + 3 files changed, 41 insertions(+) diff --git a/config.c b/config.c index 597e0460..6a398cfa 100644 --- a/config.c +++ b/config.c @@ -1096,3 +1096,36 @@ struct mddev_ident *conf_match(struct supertype *st, } return match; } + +int conf_verify_devnames(struct mddev_ident *array_list) +{ + struct mddev_ident *a1, *a2; + + for (a1 = array_list; a1; a1 = a1->next) { + if (!a1->devname) + continue; + for (a2 = a1->next; a2; a2 = a2->next) { + if (!a2->devname) + continue; + if (strcmp(a1->devname, a2->devname) != 0) + continue; + + if (a1->uuid_set && a2->uuid_set) { + char nbuf[64]; + __fname_from_uuid(a1->uuid, 0, nbuf, ':'); + fprintf(stderr, + Name ": Devices %s and ", + nbuf); + __fname_from_uuid(a2->uuid, 0, nbuf, ':'); + fprintf(stderr, + "%s have the same name: %s\n", + nbuf, a1->devname); + } else + fprintf(stderr, Name ": Device %s given twice" + " in config file\n", a1->devname); + return 1; + } + } + + return 0; +} diff --git a/mdadm.c b/mdadm.c index 8e0ce186..c6a887a8 100644 --- a/mdadm.c +++ b/mdadm.c @@ -1292,6 +1292,13 @@ int main(int argc, char *argv[]) struct map_ent *map = NULL; int cnt = 0; int failures, successes; + + if (conf_verify_devnames(array_list)) { + fprintf(stderr, Name + ": Duplicate MD device names in " + "conf file were found.\n"); + exit(1); + } if (devlist == NULL) { fprintf(stderr, Name ": No devices listed in conf file were found.\n"); exit(1); diff --git a/mdadm.h b/mdadm.h index e5dac79c..1351d420 100644 --- a/mdadm.h +++ b/mdadm.h @@ -1145,6 +1145,7 @@ extern char *conf_get_homehost(int *require_homehostp); extern char *conf_line(FILE *file); extern char *conf_word(FILE *file, int allow_key); extern int conf_name_is_free(char *name); +extern int conf_verify_devnames(struct mddev_ident *array_list); extern int devname_matches(char *name, char *match); extern struct mddev_ident *conf_match(struct supertype *st, struct mdinfo *info, -- 2.39.2