[Ocfs2-devel] ocfs2/cluster: Clean up messages in o2net
Sunil Mushran
sunil.mushran at oracle.com
Tue Aug 30 14:14:04 PDT 2011
Thanks. I'll fix the two.
On 08/25/2011 06:01 PM, Dan Carpenter wrote:
> Hello Sunil Mushran,
>
> 1dfecf810e0e: "ocfs2/cluster: Clean up messages in o2net"
> Leads to the following Smatch complaint:
> fs/ocfs2/cluster/tcp.c +1704 o2net_start_connect(101)
> error: we previously assumed 'sc' could be null (see line 1650)
>
> fs/ocfs2/cluster/tcp.c
> 1649 sc = sc_alloc(node);
> 1650 if (sc == NULL) {
> 1651 mlog(0, "couldn't allocate sc\n");
> 1652 ret = -ENOMEM;
> 1653 goto out;
> 1654 }
> 1655
>
> --- [snip] ---
>
> 1701 out:
> 1702 if (ret) {
> 1703 printk(KERN_NOTICE "o2net: Connect attempt to " SC_NODEF_FMT
> 1704 " failed with errno %d\n", SC_NODEF_ARGS(sc), ret);
> ^^
> Null dereference here.
>
> 1705 /* 0 err so that another will be queued and attempted
> 1706 * from set_nn_state */
>
> This email is from my new semi-automatic bug reporting tool script.
> Btw, Smatch complains about this as well:
>
> fs/ocfs2/cluster/tcp.c +549 o2net_set_nn_state(44)
> error: we previously assumed 'old_sc' could be null (see line 517)
>
> These dereferences were disabled in my old config, but Smatch detects
> them now that we're using normal printk()
>
> regards,
> dan carpenter
More information about the Ocfs2-devel
mailing list