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