On Tuesday June 27, [EMAIL PROTECTED] 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 [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to