[Ocfs2-devel] [patch] ocfs2: cleanup error handling in o2hb_alloc_hb_set()
Dan Carpenter
dan.carpenter at oracle.com
Mon Feb 13 12:06:43 PST 2012
On Mon, Feb 13, 2012 at 11:39:27AM -0800, Sunil Mushran wrote:
> hmm... I would say NAK because config_group_item_type_name() could
> change in the future. And there is nothing wrong with the current
> code.
The error handling isn't correct because checking "&hs->hs_group"
for NULL doesn't work. "hs" and "&hs->hs_group" are the same
address and we checked "hs" already. If we wanted to check for
allocation errors, then we would need to change the check to:
if (!hs->hs_group.cg_item.ci_name)
kfree(hs);
But that's not how the function is supposed to be used. The example
code in Documentation/filesystems/configfs/configfs_example_explicit.c
doesn't have error handling.
regards,
dan carpenter
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
Url : http://oss.oracle.com/pipermail/ocfs2-devel/attachments/20120213/de7d1584/attachment-0001.bin
More information about the Ocfs2-devel
mailing list