Hi, Please find the latest report on new defect(s) introduced to gluster/glusterfs found with Coverity Scan. 2 new defect(s) introduced to gluster/glusterfs found with Coverity Scan. 22 defect(s), reported by Coverity Scan earlier, were marked fixed in the recent build analyzed by Coverity Scan. New defect(s) Reported-by: Coverity Scan Showing 2 of 2 defect(s) ** CID 1382470: Memory - illegal accesses (BUFFER_SIZE_WARNING) /xlators/mgmt/glusterd/src/glusterd-geo-rep.c: 3227 in glusterd_op_stage_gsync_create() ________________________________________________________________________________________________________ *** CID 1382470: Memory - illegal accesses (BUFFER_SIZE_WARNING) /xlators/mgmt/glusterd/src/glusterd-geo-rep.c: 3227 in glusterd_op_stage_gsync_create() 3221 if (ret) { 3222 gf_msg (this->name, GF_LOG_ERROR, 0, GD_MSG_DICT_SET_FAILED, 3223 "Unable to store statefile path"); 3224 goto out; 3225 } 3226 >>> CID 1382470: Memory - illegal accesses (BUFFER_SIZE_WARNING) >>> Calling strncpy with a maximum size argument of 4096 bytes on destination array "statefiledir" of size 4096 bytes might leave the destination string unterminated. 3227 strncpy (statefiledir, statefile, sizeof(statefiledir)); 3228 statedir = dirname (statefiledir); 3229 3230 ret = sys_lstat (statedir, &stbuf); 3231 if (!ret && !is_force) { 3232 snprintf (errmsg, sizeof (errmsg), "Session between %s" ** CID 1382469: (DEADCODE) /heal/src/glfs-heal.c: 1749 in main() /heal/src/glfs-heal.c: 1750 in main() /heal/src/glfs-heal.c: 1751 in main() /heal/src/glfs-heal.c: 1752 in main() /heal/src/glfs-heal.c: 1753 in main() /heal/src/glfs-heal.c: 1754 in main() /heal/src/glfs-heal.c: 1755 in main() /heal/src/glfs-heal.c: 1756 in main() /heal/src/glfs-heal.c: 1757 in main() /heal/src/glfs-heal.c: 1758 in main() /heal/src/glfs-heal.c: 1759 in main() ________________________________________________________________________________________________________ *** CID 1382469: (DEADCODE) /heal/src/glfs-heal.c: 1749 in main() 1743 &rootloc, file, heal_op); 1744 break; 1745 case GF_SHD_OP_SBRAIN_HEAL_FROM_BRICK: 1746 ret = glfsh_heal_from_brick (fs, top_subvol, &rootloc, 1747 hostname, path, file); 1748 break; >>> CID 1382469: (DEADCODE) >>> Execution cannot reach this statement: "case GF_SHD_OP_INVALID:". 1749 case GF_SHD_OP_INVALID: 1750 case GF_SHD_OP_HEAL_INDEX: 1751 case GF_SHD_OP_HEAL_FULL: 1752 case GF_SHD_OP_HEALED_FILES: 1753 case GF_SHD_OP_HEAL_FAILED_FILES: 1754 case GF_SHD_OP_STATISTICS: /heal/src/glfs-heal.c: 1750 in main() 1744 break; 1745 case GF_SHD_OP_SBRAIN_HEAL_FROM_BRICK: 1746 ret = glfsh_heal_from_brick (fs, top_subvol, &rootloc, 1747 hostname, path, file); 1748 break; 1749 case GF_SHD_OP_INVALID: >>> CID 1382469: (DEADCODE) >>> Execution cannot reach this statement: "case GF_SHD_OP_HEAL_INDEX:". 1750 case GF_SHD_OP_HEAL_INDEX: 1751 case GF_SHD_OP_HEAL_FULL: 1752 case GF_SHD_OP_HEALED_FILES: 1753 case GF_SHD_OP_HEAL_FAILED_FILES: 1754 case GF_SHD_OP_STATISTICS: 1755 case GF_SHD_OP_STATISTICS_HEAL_COUNT: /heal/src/glfs-heal.c: 1751 in main() 1745 case GF_SHD_OP_SBRAIN_HEAL_FROM_BRICK: 1746 ret = glfsh_heal_from_brick (fs, top_subvol, &rootloc, 1747 hostname, path, file); 1748 break; 1749 case GF_SHD_OP_INVALID: 1750 case GF_SHD_OP_HEAL_INDEX: >>> CID 1382469: (DEADCODE) >>> Execution cannot reach this statement: "case GF_SHD_OP_HEAL_FULL:". 1751 case GF_SHD_OP_HEAL_FULL: 1752 case GF_SHD_OP_HEALED_FILES: 1753 case GF_SHD_OP_HEAL_FAILED_FILES: 1754 case GF_SHD_OP_STATISTICS: 1755 case GF_SHD_OP_STATISTICS_HEAL_COUNT: 1756 case GF_SHD_OP_STATISTICS_HEAL_COUNT_PER_REPLICA: /heal/src/glfs-heal.c: 1752 in main() 1746 ret = glfsh_heal_from_brick (fs, top_subvol, &rootloc, 1747 hostname, path, file); 1748 break; 1749 case GF_SHD_OP_INVALID: 1750 case GF_SHD_OP_HEAL_INDEX: 1751 case GF_SHD_OP_HEAL_FULL: >>> CID 1382469: (DEADCODE) >>> Execution cannot reach this statement: "case GF_SHD_OP_HEALED_FILES:". 1752 case GF_SHD_OP_HEALED_FILES: 1753 case GF_SHD_OP_HEAL_FAILED_FILES: 1754 case GF_SHD_OP_STATISTICS: 1755 case GF_SHD_OP_STATISTICS_HEAL_COUNT: 1756 case GF_SHD_OP_STATISTICS_HEAL_COUNT_PER_REPLICA: 1757 case GF_SHD_OP_HEAL_ENABLE: /heal/src/glfs-heal.c: 1753 in main() 1747 hostname, path, file); 1748 break; 1749 case GF_SHD_OP_INVALID: 1750 case GF_SHD_OP_HEAL_INDEX: 1751 case GF_SHD_OP_HEAL_FULL: 1752 case GF_SHD_OP_HEALED_FILES: >>> CID 1382469: (DEADCODE) >>> Execution cannot reach this statement: "case GF_SHD_OP_HEAL_FAILED_...". 1753 case GF_SHD_OP_HEAL_FAILED_FILES: 1754 case GF_SHD_OP_STATISTICS: 1755 case GF_SHD_OP_STATISTICS_HEAL_COUNT: 1756 case GF_SHD_OP_STATISTICS_HEAL_COUNT_PER_REPLICA: 1757 case GF_SHD_OP_HEAL_ENABLE: 1758 case GF_SHD_OP_HEAL_DISABLE: /heal/src/glfs-heal.c: 1754 in main() 1748 break; 1749 case GF_SHD_OP_INVALID: 1750 case GF_SHD_OP_HEAL_INDEX: 1751 case GF_SHD_OP_HEAL_FULL: 1752 case GF_SHD_OP_HEALED_FILES: 1753 case GF_SHD_OP_HEAL_FAILED_FILES: >>> CID 1382469: (DEADCODE) >>> Execution cannot reach this statement: "case GF_SHD_OP_STATISTICS:". 1754 case GF_SHD_OP_STATISTICS: 1755 case GF_SHD_OP_STATISTICS_HEAL_COUNT: 1756 case GF_SHD_OP_STATISTICS_HEAL_COUNT_PER_REPLICA: 1757 case GF_SHD_OP_HEAL_ENABLE: 1758 case GF_SHD_OP_HEAL_DISABLE: 1759 case GF_SHD_OP_GRANULAR_ENTRY_HEAL_DISABLE: /heal/src/glfs-heal.c: 1755 in main() 1749 case GF_SHD_OP_INVALID: 1750 case GF_SHD_OP_HEAL_INDEX: 1751 case GF_SHD_OP_HEAL_FULL: 1752 case GF_SHD_OP_HEALED_FILES: 1753 case GF_SHD_OP_HEAL_FAILED_FILES: 1754 case GF_SHD_OP_STATISTICS: >>> CID 1382469: (DEADCODE) >>> Execution cannot reach this statement: "case GF_SHD_OP_STATISTICS_H...". 1755 case GF_SHD_OP_STATISTICS_HEAL_COUNT: 1756 case GF_SHD_OP_STATISTICS_HEAL_COUNT_PER_REPLICA: 1757 case GF_SHD_OP_HEAL_ENABLE: 1758 case GF_SHD_OP_HEAL_DISABLE: 1759 case GF_SHD_OP_GRANULAR_ENTRY_HEAL_DISABLE: 1760 ret = -EINVAL; /heal/src/glfs-heal.c: 1756 in main() 1750 case GF_SHD_OP_HEAL_INDEX: 1751 case GF_SHD_OP_HEAL_FULL: 1752 case GF_SHD_OP_HEALED_FILES: 1753 case GF_SHD_OP_HEAL_FAILED_FILES: 1754 case GF_SHD_OP_STATISTICS: 1755 case GF_SHD_OP_STATISTICS_HEAL_COUNT: >>> CID 1382469: (DEADCODE) >>> Execution cannot reach this statement: "case GF_SHD_OP_STATISTICS_H...". 1756 case GF_SHD_OP_STATISTICS_HEAL_COUNT_PER_REPLICA: 1757 case GF_SHD_OP_HEAL_ENABLE: 1758 case GF_SHD_OP_HEAL_DISABLE: 1759 case GF_SHD_OP_GRANULAR_ENTRY_HEAL_DISABLE: 1760 ret = -EINVAL; 1761 break; /heal/src/glfs-heal.c: 1757 in main() 1751 case GF_SHD_OP_HEAL_FULL: 1752 case GF_SHD_OP_HEALED_FILES: 1753 case GF_SHD_OP_HEAL_FAILED_FILES: 1754 case GF_SHD_OP_STATISTICS: 1755 case GF_SHD_OP_STATISTICS_HEAL_COUNT: 1756 case GF_SHD_OP_STATISTICS_HEAL_COUNT_PER_REPLICA: >>> CID 1382469: (DEADCODE) >>> Execution cannot reach this statement: "case GF_SHD_OP_HEAL_ENABLE:". 1757 case GF_SHD_OP_HEAL_ENABLE: 1758 case GF_SHD_OP_HEAL_DISABLE: 1759 case GF_SHD_OP_GRANULAR_ENTRY_HEAL_DISABLE: 1760 ret = -EINVAL; 1761 break; 1762 } /heal/src/glfs-heal.c: 1758 in main() 1752 case GF_SHD_OP_HEALED_FILES: 1753 case GF_SHD_OP_HEAL_FAILED_FILES: 1754 case GF_SHD_OP_STATISTICS: 1755 case GF_SHD_OP_STATISTICS_HEAL_COUNT: 1756 case GF_SHD_OP_STATISTICS_HEAL_COUNT_PER_REPLICA: 1757 case GF_SHD_OP_HEAL_ENABLE: >>> CID 1382469: (DEADCODE) >>> Execution cannot reach this statement: "case GF_SHD_OP_HEAL_DISABLE:". 1758 case GF_SHD_OP_HEAL_DISABLE: 1759 case GF_SHD_OP_GRANULAR_ENTRY_HEAL_DISABLE: 1760 ret = -EINVAL; 1761 break; 1762 } 1763 /heal/src/glfs-heal.c: 1759 in main() 1753 case GF_SHD_OP_HEAL_FAILED_FILES: 1754 case GF_SHD_OP_STATISTICS: 1755 case GF_SHD_OP_STATISTICS_HEAL_COUNT: 1756 case GF_SHD_OP_STATISTICS_HEAL_COUNT_PER_REPLICA: 1757 case GF_SHD_OP_HEAL_ENABLE: 1758 case GF_SHD_OP_HEAL_DISABLE: >>> CID 1382469: (DEADCODE) >>> Execution cannot reach this statement: "case GF_SHD_OP_GRANULAR_ENT...". 1759 case GF_SHD_OP_GRANULAR_ENTRY_HEAL_DISABLE: 1760 ret = -EINVAL; 1761 break; 1762 } 1763 1764 glfsh_output->end (ret, NULL); ________________________________________________________________________________________________________ To view the defects in Coverity Scan visit, https://u2389337.ct.sendgrid.net/wf/click?upn=08onrYu34A-2BWcWUl-2F-2BfV0V05UPxvVjWch-2Bd2MGckcRZBK54bFWohdObZ6wlkeK264nDC24cnLwH4MTOSDXRjQcO27-2F6DmQXPB4g4Mz-2BEJJ0-3D_pIB5qNyu3krz5eLOmjwFZkKK-2FJ3CPgtiU1pdCCu-2BphdokdXKV3dj-2BwRLX-2BbNrO-2FgxEox5ySxXvt6Cw2yRH-2FMJOxJUzzpm2DIArT3ehUWnruiLgZdxdTw-2FUnmUe8DS5-2FkXE00VpaymlTtoZB9fwdIn2RXdnl01EziBHDZRxse9x3MTGsvN1MFsmMlCMJJDogSlLw3aegZMvZKg1bDsDgK5-2FJkJ0aQhmKXs6bdrF92Wfw-3D To manage Coverity Scan email notifications for "gluster-devel@xxxxxxxxxxx", click https://u2389337.ct.sendgrid.net/wf/click?upn=08onrYu34A-2BWcWUl-2F-2BfV0V05UPxvVjWch-2Bd2MGckcRbVDbis712qZDP-2FA8y06Nq48r9wNv8-2BLR0R-2BBlXm7ottYIbnE5UU-2F0Z00EseY5i5N2c1DtvQJ8Cp-2BSp5bYUPGR1MbiG-2F6KsJiGF2zIdiuCrkCwl5z0BqidHVuHFKKiyVZo-3D_pIB5qNyu3krz5eLOmjwFZkKK-2FJ3CPgtiU1pdCCu-2BphdokdXKV3dj-2BwRLX-2BbNrO-2FgxEox5ySxXvt6Cw2yRH-2FMJNKD9PHMCWETnU7KzKBg9p6GaKaGguBv-2Bi8RlwCvos1pDCkppLmo2atRX-2BkFRQV-2BzByM-2F-2B7s4SSu5pJhyTF0qWhcVtSJjT-2FjXRNQsyK0YbwJ5CRLF-2B5w6g224DpFIfi7dkotezGas7RYwdubkJh1PYI-3D _______________________________________________ Gluster-devel mailing list Gluster-devel@xxxxxxxxxxx http://lists.gluster.org/mailman/listinfo/gluster-devel