On Tuesday June 27, nix@xxxxxxxxxxxxx wrote:
> > And the code in load_partitions() handles a NULL return.
>
> Er, what?
>
> The code in load_partitions() in mdadm 2.5.1 and below says
>
> ,----[ config.c:load_partitions() ]
> | name = map_dev(major, minor, 1);
> |
> | d = malloc(sizeof(*d));
> | d->devname = strdup(name);
> `----
>
Ahh.. uhmmm... Oh yes. I've fixed that since, but completely forgot
about it and the change log didn't make it obvious.
From
http://neil.brown.name/git/?p=mdadm
28 hours ago Neil Brown More consistent honoring of --configfile
28 hours ago Neil Brown Fix user-after-free bug in error path in --monitor ...
28 hours ago Neil Brown Improve ftw handling.
31 hours ago Neil Brown Fix a recently introduced bug, and make --assemble ...
31 hours ago Neil Brown Work around bug in --add handling for version-1 superb ...
31 hours ago Neil Brown Add 'Array Slot' line to --examine for version-1 superb ...
31 hours ago martin f krafft Fix manpage typo
31 hours ago Neil Brown Get the changelog uptodate for recent changes.
7 days ago Paul Clements Move a variable declaration to the declaration area.
this one-> 7 days ago Neil Brown Stop map_dev from returning [0:0]
7 days ago Paul Clements Fix problem with post-increment usage in macro
11 days ago Neil Brown Release 2.5.1
So: that's fixed for 2.5.2. Thanks for following it up.
NeilBrown
-
To unsubscribe from this list: send the line "unsubscribe linux-raid" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
