[Ocfs2-tools-devel] [PATCH] fsck.ocfs2: fix a minor memleak

Sunil Mushran sunil.mushran at oracle.com
Thu Nov 5 17:50:23 PST 2009


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);



More information about the Ocfs2-tools-devel mailing list