Tristan wrote: > > I guess memory should not be placed there, it should be following instead: > > else { > > ret = OCFS2_ET_INTERNAL_FAILURE; > goto out; > > } > > out: > if (dp) > free(dp); > ... > I would imagine you mean.... out: if (ret && dp) free(dp);