From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bob Peterson Date: Wed, 13 Nov 2019 15:30:22 -0600 Subject: [Cluster-devel] [PATCH 24/32] gfs2: initialize tr_ail1_list when creating transactions In-Reply-To: <20191113213030.237431-1-rpeterso@redhat.com> References: <20191113213030.237431-1-rpeterso@redhat.com> Message-ID: <20191113213030.237431-25-rpeterso@redhat.com> List-Id: To: cluster-devel.redhat.com MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit In function gfs2_trans_begin, new transactions were created but their ail1 list, tr_ail1_list was never initialized. Therefore it relied upon other circumstances when the list became empty. This patch adds proper initialization of the list. Signed-off-by: Bob Peterson --- fs/gfs2/trans.c | 1 + 1 file changed, 1 insertion(+) diff --git a/fs/gfs2/trans.c b/fs/gfs2/trans.c index 7d16d9aa3153..790fdd4e8c5e 100644 --- a/fs/gfs2/trans.c +++ b/fs/gfs2/trans.c @@ -58,6 +58,7 @@ int gfs2_trans_begin(struct gfs2_sbd *sdp, unsigned int blocks, sizeof(u64)); gfs2_assert_warn(sdp, list_empty(&tr->tr_databuf)); gfs2_assert_warn(sdp, list_empty(&tr->tr_buf)); + INIT_LIST_HEAD(&tr->tr_ail1_list); sb_start_intwrite(sdp->sd_vfs); -- 2.23.0