On Thu, Jan 10, 2019 at 02:06:38PM +0800, zhangyi (F) wrote:
> The jh pointer may be used uninitialized in the two cases below and the
> compiler complain about it when enabling JBUFFER_TRACE macro, fix them.
>
> In file included from fs/jbd2/transaction.c:19:0:
> fs/jbd2/transaction.c: In function ‘jbd2_journal_get_undo_access’:
> ./include/linux/jbd2.h:1637:38: warning: ‘jh’ is used uninitialized in this function [-Wuninitialized]
> #define JBUFFER_TRACE(jh, info) do { printk("%s: %d\n", __func__, jh->b_jcount);} while (0)
> ^
> fs/jbd2/transaction.c:1219:23: note: ‘jh’ was declared here
> struct journal_head *jh;
> ^
> In file included from fs/jbd2/transaction.c:19:0:
> fs/jbd2/transaction.c: In function ‘jbd2_journal_dirty_metadata’:
> ./include/linux/jbd2.h:1637:38: warning: ‘jh’ may be used uninitialized in this function [-Wmaybe-uninitialized]
> #define JBUFFER_TRACE(jh, info) do { printk("%s: %d\n", __func__, jh->b_jcount);} while (0)
> ^
> fs/jbd2/transaction.c:1332:23: note: ‘jh’ was declared here
> struct journal_head *jh;
> ^
>
> Signed-off-by: zhangyi (F) <yi.zhang@xxxxxxxxxx>
> Cc: stable@xxxxxxxxxxxxxxx
Thanks, applied.
- Ted
