[Ocfs2-tools-devel] [PATCH 4/7] libocfs2: add two types of trees operation of xattr

[Tiger Yang]

Tristan wrote:
> I suddenly hit a bug there when applying your patches to do the xattr 
> corruptions,
> 
> Actually, the function ptr 'eo_get_clusters' you used here is NULL, 
> refer to ocfs2_dinode_et_opts's initialization in /libocfs2/extent_tree.c:
> 
> static struct ocfs2_extent_tree_operations ocfs2_dinode_et_ops = {
> .eo_set_last_eb_blk = ocfs2_dinode_set_last_eb_blk,
> .eo_get_last_eb_blk = ocfs2_dinode_get_last_eb_blk,
> .eo_update_clusters = ocfs2_dinode_update_clusters,
> .eo_fill_root_el = ocfs2_dinode_fill_root_el,
> };
> 
> As you seen, memeber 'eo_get_clusters' didn't get initialized here.
> 
yes, this is a big bug will fix it in the next version.

thanks,
tiger