diff options
| author | David Woodhouse <dwmw2@infradead.org> | 2007-12-03 12:46:12 +0000 |
|---|---|---|
| committer | David Woodhouse <dwmw2@infradead.org> | 2007-12-03 12:46:12 +0000 |
| commit | ce37ab42ad8b38ef2f36c31c6b4c39b87f36b792 (patch) | |
| tree | 6d050ed72e3044401008e8c3e3ae60adc68be4d9 | |
| parent | 846fc31d06e54ad94026da11da0668c050fe777e (diff) | |
[MTD] Always initialise mutex in new mtd_blktrans_dev.
We were only initialising the mutex in the case where the new device was
automatically allocated the highest minor number. If the caller
specified a minor number, or if it filled in a free slot which was made
by a previous device deregistering, the mutex wouldn't get initialised
when we jumped out of the loop.
Reported by Monte Copeland <catboat@texas.net>
Signed-off-by: David Woodhouse <dwmw2@infradead.org>
| -rw-r--r-- | drivers/mtd/mtd_blkdevs.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/mtd/mtd_blkdevs.c b/drivers/mtd/mtd_blkdevs.c index 74d9d30edab..839eed8430a 100644 --- a/drivers/mtd/mtd_blkdevs.c +++ b/drivers/mtd/mtd_blkdevs.c @@ -248,9 +248,9 @@ int add_mtd_blktrans_dev(struct mtd_blktrans_dev *new) return -EBUSY; } - mutex_init(&new->lock); list_add_tail(&new->list, &tr->devs); added: + mutex_init(&new->lock); if (!tr->writesect) new->readonly = 1; |