<html>
<head>
<meta http-equiv="content-type" content="text/html; charset=UTF-8">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<br>
<div class="moz-forward-container">To fix these Coverity issues ,
please check the below link for guidelines:<br>
<br>
<a class="moz-txt-link-freetext"
href="http://www.gluster.org/community/documentation/index.php/Fixing_Issues_Reported_By_Tools_For_Static_Code_Analysis#Coverity">http://www.gluster.org/community/documentation/index.php/Fixing_Issues_Reported_By_Tools_For_Static_Code_Analysis#Coverity</a><br>
<br>
-------- Original Message --------
<table class="moz-email-headers-table" border="0" cellpadding="0"
cellspacing="0">
<tbody>
<tr>
<th align="RIGHT" nowrap="nowrap" valign="BASELINE">Subject:
</th>
<td>New Defects reported by Coverity Scan for GlusterFS</td>
</tr>
<tr>
<th align="RIGHT" nowrap="nowrap" valign="BASELINE">Date: </th>
<td>Thu, 11 Sep 2014 00:02:11 -0700</td>
</tr>
<tr>
<th align="RIGHT" nowrap="nowrap" valign="BASELINE">From: </th>
<td><a class="moz-txt-link-abbreviated" href="mailto:scan-admin@coverity.com">scan-admin@coverity.com</a></td>
</tr>
</tbody>
</table>
<br>
<br>
<pre>Hi,
Please find the latest report on new defect(s) introduced to GlusterFS found with Coverity Scan.
Defect(s) Reported-by: Coverity Scan
Showing 4 of 4 defect(s)
** CID 1238186: Logically dead code (DEADCODE)
/xlators/cluster/afr/src/afr-dir-write.c: 339 in afr_mark_new_entry_changelog()
** CID 1238185: Explicit null dereferenced (FORWARD_NULL)
/xlators/features/snapview-server/src/snapview-server-mgmt.c: 476 in svs_get_snapshot_list()
** CID 1238184: Explicit null dereferenced (FORWARD_NULL)
/xlators/features/snapview-server/src/snapview-server-mgmt.c: 115 in svs_mgmt_init()
** CID 1238183: Missing break in switch (MISSING_BREAK)
/xlators/mgmt/glusterd/src/glusterd-rebalance.c: 577 in glusterd_op_stage_rebalance()
________________________________________________________________________________________________________
*** CID 1238186: Logically dead code (DEADCODE)
/xlators/cluster/afr/src/afr-dir-write.c: 339 in afr_mark_new_entry_changelog()
333 break;
334 }
335
336 new_frame = NULL;
337 out:
338 if (changelog)
>>> CID 1238186: Logically dead code (DEADCODE)
>>> Execution cannot reach this statement "afr_matrix_cleanup(changelo...".
339 afr_matrix_cleanup (changelog, priv->child_count);
340 if (new_frame)
341 AFR_STACK_DESTROY (new_frame);
342 if (xattr)
343 dict_unref (xattr);
344 return;
________________________________________________________________________________________________________
*** CID 1238185: Explicit null dereferenced (FORWARD_NULL)
/xlators/features/snapview-server/src/snapview-server-mgmt.c: 476 in svs_get_snapshot_list()
470 if (frame_cleanup) {
471 /*
472 * Destroy the frame if we encountered an error
473 * Else we need to clean it up in
474 * mgmt_get_snapinfo_cbk
475 */
>>> CID 1238185: Explicit null dereferenced (FORWARD_NULL)
>>> Dereferencing null pointer "frame".
476 SVS_STACK_DESTROY (frame);
477 }
478
479 return ret;
________________________________________________________________________________________________________
*** CID 1238184: Explicit null dereferenced (FORWARD_NULL)
/xlators/features/snapview-server/src/snapview-server-mgmt.c: 115 in svs_mgmt_init()
109 ret = 0;
110
111 gf_log (this->name, GF_LOG_DEBUG, "svs mgmt init successful");
112
113 out:
114 if (ret) {
>>> CID 1238184: Explicit null dereferenced (FORWARD_NULL)
>>> Dereferencing null pointer "priv".
115 rpc_clnt_connection_cleanup (&priv->rpc->conn);
116 rpc_clnt_unref (priv->rpc);
117 priv->rpc = NULL;
118 }
119
120 return ret;
________________________________________________________________________________________________________
*** CID 1238183: Missing break in switch (MISSING_BREAK)
/xlators/mgmt/glusterd/src/glusterd-rebalance.c: 577 in glusterd_op_stage_rebalance()
571 "disconnect those clients before "
572 "attempting this command again.",
573 volname);
574 goto out;
575 }
576
>>> CID 1238183: Missing break in switch (MISSING_BREAK)
>>> The above case falls through to this one.
577 case GF_DEFRAG_CMD_START_FORCE:
578 if (is_origin_glusterd (dict)) {
579 op_ctx = glusterd_op_get_ctx ();
580 if (!op_ctx) {
581 ret = -1;
582 gf_log (this->name, GF_LOG_ERROR,
________________________________________________________________________________________________________
To view the defects in Coverity Scan visit, <a class="moz-txt-link-freetext" href="http://scan.coverity.com/projects/987?tab=overview">http://scan.coverity.com/projects/987?tab=overview</a>
To unsubscribe from the email notification for new defects, <a class="moz-txt-link-freetext" href="http://scan5.coverity.com/cgi-bin/unsubscribe.py">http://scan5.coverity.com/cgi-bin/unsubscribe.py</a>
</pre>
<br>
</div>
<br>
</body>
</html>