diff options
| author | Artem Bityutskiy <Artem.Bityutskiy@nokia.com> | 2007-05-21 13:26:05 -0400 |
|---|---|---|
| committer | Artem Bityutskiy <Artem.Bityutskiy@nokia.com> | 2007-07-18 09:53:57 -0400 |
| commit | a443db48e05a8d2d8db0a17409655c58da65a35e (patch) | |
| tree | becf7ff602745c657d2d07a7354ba7cca0fe20c7 /drivers | |
| parent | 3261ebd7d4194ff30d0eae7ba8d937dcccf7235d (diff) | |
UBI: bugfix in ubi_leb_change()
Do not call 'ubi_wl_put_peb()' if the LEB was unmapped.
Reported-by: Gabor Loki <loki@inf.u-szeged.hu>
Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/mtd/ubi/eba.c | 12 |
1 files changed, 7 insertions, 5 deletions
diff --git a/drivers/mtd/ubi/eba.c b/drivers/mtd/ubi/eba.c index a1820151e9f9..0db8dc7af8a0 100644 --- a/drivers/mtd/ubi/eba.c +++ b/drivers/mtd/ubi/eba.c | |||
| @@ -891,11 +891,13 @@ retry: | |||
| 891 | goto write_error; | 891 | goto write_error; |
| 892 | } | 892 | } |
| 893 | 893 | ||
| 894 | err = ubi_wl_put_peb(ubi, vol->eba_tbl[lnum], 1); | 894 | if (vol->eba_tbl[lnum] >= 0) { |
| 895 | if (err) { | 895 | err = ubi_wl_put_peb(ubi, vol->eba_tbl[lnum], 1); |
| 896 | ubi_free_vid_hdr(ubi, vid_hdr); | 896 | if (err) { |
| 897 | leb_write_unlock(ubi, vol_id, lnum); | 897 | ubi_free_vid_hdr(ubi, vid_hdr); |
| 898 | return err; | 898 | leb_write_unlock(ubi, vol_id, lnum); |
| 899 | return err; | ||
| 900 | } | ||
| 899 | } | 901 | } |
| 900 | 902 | ||
| 901 | vol->eba_tbl[lnum] = pnum; | 903 | vol->eba_tbl[lnum] = pnum; |