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

[Vijay Bellur]

On 12/17/2013 11:17 AM, Lalatendu Mohanty wrote:
> -------- Original Message --------
> Subject:        New Defects reported by Coverity Scan for GlusterFS
> Date:   Mon, 16 Dec 2013 12:49:25 -0800
> From:   address@hidden
>
>
>
> Hi,
>
>
> Please find the latest report on new defect(s) introduced to GlusterFS found 
> with Coverity Scan.
>
> Defect(s) Reported-by: Coverity Scan
> Showing 7 of 8 defect(s)
>
>
> ** CID 1138525:  Missing break in switch  (MISSING_BREAK)
> /xlators/mgmt/glusterd/src/glusterd-quota.c: 1407 in glusterd_op_stage_quota()

Raghavendra: Can you please confirm if this is an intentional "missing 
break"?

> ** CID 1138522:  Destination buffer too small  (BUFFER_SIZE)
> /xlators/mgmt/glusterd/src/glusterd-geo-rep.c: 2926 in 
> glusterd_read_status_file()
> /xlators/mgmt/glusterd/src/glusterd-geo-rep.c: 2928 in 
> glusterd_read_status_file()
> /xlators/mgmt/glusterd/src/glusterd-geo-rep.c: 2904 in 
> glusterd_read_status_file()
>
> ** CID 1138523:  Logically dead code  (DEADCODE)
> /xlators/mgmt/glusterd/src/glusterd-geo-rep.c: 2634 in 
> glusterd_parse_gsync_status()
> /xlators/mgmt/glusterd/src/glusterd-geo-rep.c: 2654 in 
> glusterd_parse_gsync_status()
> /xlators/mgmt/glusterd/src/glusterd-geo-rep.c: 2668 in 
> glusterd_parse_gsync_status()
> /xlators/mgmt/glusterd/src/glusterd-geo-rep.c: 2624 in 
> glusterd_parse_gsync_status()
> /xlators/mgmt/glusterd/src/glusterd-geo-rep.c: 2630 in 
> glusterd_parse_gsync_status()
> /xlators/mgmt/glusterd/src/glusterd-geo-rep.c: 2650 in 
> glusterd_parse_gsync_status()
> /xlators/mgmt/glusterd/src/glusterd-geo-rep.c: 2664 in 
> glusterd_parse_gsync_status()
>
> ** CID 1138524:  Explicit null dereferenced  (FORWARD_NULL)
> /xlators/mgmt/glusterd/src/glusterd-geo-rep.c: 2679 in 
> glusterd_parse_gsync_status()
>
> ** CID 1138527:  Resource leak  (RESOURCE_LEAK)
> /xlators/mgmt/glusterd/src/glusterd-utils.c: 4750 in _local_gsyncd_start()
>
> ** CID 1138526:  Resource leak  (RESOURCE_LEAK)
> /xlators/mgmt/glusterd/src/glusterd-geo-rep.c: 2683 in 
> glusterd_parse_gsync_status()

Ajeet, Avra: Can you please look into these defects?

Thanks,
Vijay