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 <lukasz.orlowski@xxxxxxxxx> --- config.c | 25 +++++++++++++++++++++++++ mdadm.c | 7 +++++++ mdadm.h | 1 + 3 files changed, 33 insertions(+), 0 deletions(-) diff --git a/config.c b/config.c index c0a6baa..9b337eb 100644 --- a/config.c +++ b/config.c @@ -1088,3 +1088,28 @@ struct mddev_ident *conf_match(struct mdinfo *info, 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)) { + fprintf(stderr, + Name ": Devices %x:%x:%x:%x and " + "%x:%x:%x:%x have the same name: %s\n", + a1->uuid[0], a1->uuid[1], a1->uuid[2], + a1->uuid[3], a2->uuid[0], a2->uuid[1], + a2->uuid[2], a2->uuid[3], a1->devname); + return 1; + } + } + } + + return 0; +} diff --git a/mdadm.c b/mdadm.c index 56de7b7..ebf4c4d 100644 --- a/mdadm.c +++ b/mdadm.c @@ -1282,6 +1282,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 6dbc5ea..bff454f 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 mdinfo *info, struct supertype *st); extern int experimental(void); --------------------------------------------------------------------- Intel Technology Poland sp. z o.o. z siedziba w Gdansku ul. Slowackiego 173 80-298 Gdansk Sad Rejonowy Gdansk Polnoc w Gdansku, VII Wydzial Gospodarczy Krajowego Rejestru Sadowego, numer KRS 101882 NIP 957-07-52-316 Kapital zakladowy 200.000 zl This e-mail and any attachments may contain confidential material for the sole use of the intended recipient(s). Any review or distribution by others is strictly prohibited. If you are not the intended recipient, please contact the sender and delete all copies. ��.n��������+%������w��{.n�����{����w��ܨ}���Ơz�j:+v�����w����ޙ��&�)ߡ�a����z�ޗ���ݢj��w�f