diff options
| author |   Zhang Qiang <zhangqiang.buaa@gmail.com> | 2009-08-09 19:13:10 +0800 |
|---|---|---|
| committer |   Ryusuke Konishi <konishi.ryusuke@lab.ntt.co.jp> | 2009-09-14 18:27:14 +0900 |
| commit | a4f0b9c5b4ae83636dafde8f3a0e04b5e411a0f3 (patch) | |
| tree | 885111c7beec6e69680768c7fb1a8b22d53cd175 | |
| parent | nilfs2: use GFP_NOIO for bio_alloc instead of GFP_NOWAIT (diff) | |
| download | linux-dev-a4f0b9c5b4ae83636dafde8f3a0e04b5e411a0f3.tar.xz linux-dev-a4f0b9c5b4ae83636dafde8f3a0e04b5e411a0f3.zip | |
nilfs2: An unassigned variable is assigned to a never used structure member
nilfs2: In procedure 'nilfs_get_sb()', when a nilfs filesysttem is
mounted for the first time, local variable 'nilfs->ns_last_cno' is
used before loading the latest checkpoint number from disk (in
'nilfs_fill_super'). 'nilfs->ns_last_cno' is assigned to 'sd.cno', but
'sd.cno' has never been used in the procedure.
Signed-off-by: Zhang Qiang <zhangqiang.buaa@gmail.com>
Signed-off-by: Ryusuke Konishi <konishi.ryusuke@lab.ntt.co.jp>
| -rw-r--r-- | fs/nilfs2/super.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/fs/nilfs2/super.c b/fs/nilfs2/super.c index ed8ec5718ae6..019752f7d2da 100644 --- a/fs/nilfs2/super.c +++ b/fs/nilfs2/super.c @@ -1103,10 +1103,6 @@ nilfs_get_sb(struct file_system_type *fs_type, int flags, */ sd.sbi = nilfs_find_sbinfo(nilfs, !(flags & MS_RDONLY), sd.cno); - if (!sd.cno) - /* trying to get the latest checkpoint. */ - sd.cno = nilfs_last_cno(nilfs); - /* * Get super block instance holding the nilfs_sb_info struct. * A new instance is allocated if no existing mount is present or |