[PATCH] ext3: fix list scanning in __cleanup_transaction
authorJan Kara <jack@suse.cz>
Thu, 2 Jun 2005 21:02:01 +0000 (14:02 -0700)
committerLinus Torvalds <torvalds@ppc970.osdl.org>
Thu, 2 Jun 2005 22:12:29 +0000 (15:12 -0700)
commit7e3b11a9be6ac94bf4af81757b6a10e7e65b846f
tree6159b542abf0515e5980822546ca0f84380cd3d3
parent00ea81459c279f14a7b344320a71c94f60f88929
[PATCH] ext3: fix list scanning in __cleanup_transaction

Fix a bug in list scanning that can cause us to skip the last buffer on the
checkpoint list (and hence fail to do any progress under some rather
unfavorable conditions).

The problem is we first do jh=next_jh and then test

} while (jh!=last_jh);

Hence we skip the last buffer on the list (if it was not the only buffer on
the list).  As we already do jh=next_jh; in the beginning of the loop we
are safe to just remove the assignment in the end.  It can happen that 'jh'
will be freed at the point we test jh != last_jh but that does not matter
as we never *dereference* the pointer.

Signed-off-by: Jan Kara <jack@suse.cz>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
fs/jbd/checkpoint.c