Hi, I was trying to understand more about LDLM, specifically Lock Value Block (LVB's) and stumbled upon the following code block : static int mdc_resource_inode_free(struct ldlm_resource *res) { if (res->lr_lvb_inode) res->lr_lvb_inode = NULL; return 0; } Shouldn't the pointer (lr_lvb_inode) be freed before assigning it to NULL ? Just curious to know. -- Regards Sudheendra Sampath