diff options
author | Lukasz Dorau <lukasz.dorau@intel.com> | 2011-10-02 23:55:02 +0200 |
---|---|---|
committer | NeilBrown <neilb@suse.de> | 2011-10-02 23:55:02 +0200 |
commit | cc700db34f6fb565b37f4edf7fe7fe40a5f2745b (patch) | |
tree | e66823392910fe75f39af2fcb624ae10d68fced0 /mapfile.c | |
parent | Fix small memory leak (diff) | |
download | mdadm-cc700db34f6fb565b37f4edf7fe7fe40a5f2745b.tar.xz mdadm-cc700db34f6fb565b37f4edf7fe7fe40a5f2745b.zip |
fix: correct unlocking of map file
1. Three missing map_unlock() calls were added.
2. Map file must be unlocked on fork, else child will hold lock.
Signed-off-by: Lukasz Dorau <lukasz.dorau@intel.com>
Signed-off-by: NeilBrown <neilb@suse.de>
Diffstat (limited to 'mapfile.c')
-rw-r--r-- | mapfile.c | 12 |
1 files changed, 12 insertions, 0 deletions
@@ -159,6 +159,18 @@ void map_unlock(struct map_ent **melp) lf = NULL; } +void map_fork(void) +{ + /* We are forking, so must close the lock file. + * Don't risk flushing anything though. + */ + if (lf) { + close(fileno(lf)); + fclose(lf); + lf = NULL; + } +} + void map_add(struct map_ent **melp, int devnum, char *metadata, int uuid[4], char *path) { |