[Gluster-devel] Fwd: New Defects reported by Coverity Scan for GlusterFS

[Lalatendu Mohanty]

On 04/03/2014 10:19 AM, Atin Mukherjee wrote:
> Hi Lala,
>
> This has been introduced by my patch  : http://review.gluster.org/7292
> I will fix it in sometime.
>
> Regards,
> Atin Mukherjee
> Senior Software Engineer
> E-9, Extn : 73196, Direct : 08039245196, Mobile: +919739491377
> Email - amukherj at redhat.com

Thanks Atin. Here is the wiki page for Coverity and other static 
analysis tools to help you get started.

http://www.gluster.org/community/documentation/index.php/Fixing_Issues_Reported_By_Tools_For_Static_Code_Analysis

-Lala
>
> ------------------------------------------------------------------------
> *From: *"Lalatendu Mohanty" <lmohanty at redhat.com>
> *To: *gluster-devel at nongnu.org
> *Sent: *Thursday, April 3, 2014 8:31:37 AM
> *Subject: *[Gluster-devel] Fwd: New Defects reported by Coverity Scan 
> for        GlusterFS
>
>
>
>
> -------- Original Message --------
> Subject: 	New Defects reported by Coverity Scan for GlusterFS
> Date: 	Wed, 02 Apr 2014 13:22:24 -0700
> From: 	scan-admin at coverity.com
>
>
>
> Hi,
>
>
> Please find the latest report on new defect(s) introduced to GlusterFS found with Coverity Scan.
>
> Defect(s) Reported-by: Coverity Scan
> Showing 1 of 1 defect(s)
>
>
> ** CID 1195423:  Logically dead code  (DEADCODE)
> /cli/src/cli-cmd-parser.c: 1188 in cli_cmd_volume_remove_brick_parse()
>
>
> ________________________________________________________________________________________________________
> *** CID 1195423:  Logically dead code  (DEADCODE)
> /cli/src/cli-cmd-parser.c: 1188 in cli_cmd_volume_remove_brick_parse()
> 1182                             ret = -1;
> 1183                             goto out;
> 1184                     }
> 1185             }
> 1186
> 1187             if (wordcount < 4) {
> >>>     CID 1195423:  Logically dead code  (DEADCODE)
> >>>     Execution cannot reach this statement "ret = -1;".
> 1188                     ret = -1;
> 1189                     goto out;
> 1190             }
> 1191
> 1192             ret = dict_set_int32 (dict, "command", command);
> 1193             if (ret)
>
>
> ________________________________________________________________________________________________________
> To view the defects in Coverity Scan visit,http://scan.coverity.com/projects/987?tab=Overview
>
> To unsubscribe from the email notification for new defects,http://scan5.coverity.com/cgi-bin/unsubscribe.py
>
>
>
>
>
>
> _______________________________________________
> Gluster-devel mailing list
> Gluster-devel at nongnu.org
> https://lists.nongnu.org/mailman/listinfo/gluster-devel
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://supercolony.gluster.org/pipermail/gluster-devel/attachments/20140403/4581b9f3/attachment.html>