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
_______________________________________________ Ocfs2-devel mailing list Ocfs2-devel@oss.oracle.com http://oss.oracle.com/mailman/listinfo/ocfs2-devel