DDF: fix possible mdmon crash when updating metadata.
Testing 'c' and then using 'vdc' assumes that the two are in sync, but sometimes they aren't. Testing 'vdc' is safer. This avoids a crash in some cases when failing/removing/added devices to a DDF. Signed-off-by: NeilBrown <neilb@suse.de>
This commit is contained in:
parent
a44e993e37
commit
188d31ed2b
|
@ -3022,7 +3022,7 @@ static int __write_ddf_structure(struct dl *d, struct ddf_super *ddf, __u8 type)
|
||||||
(const struct vd_config **)&vdc,
|
(const struct vd_config **)&vdc,
|
||||||
&dummy);
|
&dummy);
|
||||||
}
|
}
|
||||||
if (c) {
|
if (vdc) {
|
||||||
dprintf("writing conf record %i on disk %08x for %s/%u\n",
|
dprintf("writing conf record %i on disk %08x for %s/%u\n",
|
||||||
i, be32_to_cpu(d->disk.refnum),
|
i, be32_to_cpu(d->disk.refnum),
|
||||||
guid_str(vdc->guid),
|
guid_str(vdc->guid),
|
||||||
|
|
Loading…
Reference in New Issue