From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Tue, 26 Jan 2021 11:41:52 -0500 Subject: [scan-admin@coverity.com: New Defects reported by Coverity Scan for Das U-Boot] Message-ID: <20210126164152.GV7530@bill-the-cat> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de One new issue since the last time I ran this, and I think after reading the code myself, argc counting / sanity checking should be handled a little more clearly as well. I'm going to take a quick attempt at updating this. ----- Forwarded message from scan-admin at coverity.com ----- Date: Tue, 26 Jan 2021 14:49:09 +0000 (UTC) From: scan-admin@coverity.com To: tom.rini at gmail.com Subject: New Defects reported by Coverity Scan for Das U-Boot Hi, Please find the latest report on new defect(s) introduced to Das U-Boot found with Coverity Scan. 1 new defect(s) introduced to Das U-Boot found with Coverity Scan. 11 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 1 of 1 defect(s) ** CID 316601: Control flow issues (DEADCODE) /cmd/pwm.c: 76 in do_pwm() ________________________________________________________________________________________________________ *** CID 316601: Control flow issues (DEADCODE) /cmd/pwm.c: 76 in do_pwm() 70 if (argc > 0) { 71 str_channel = *argv; 72 channel = simple_strtoul(str_channel, NULL, 10); 73 argc--; 74 argv++; 75 } else { >>> CID 316601: Control flow issues (DEADCODE) >>> Execution cannot reach this statement: "return CMD_RET_USAGE;". 76 return CMD_RET_USAGE; 77 } 78 79 if (sub_cmd == PWM_SET_INVERT && argc > 0) { 80 str_enable = *argv; 81 pwm_enable = simple_strtoul(str_enable, NULL, 10); ________________________________________________________________________________________________________ To view the defects in Coverity Scan visit, https://u15810271.ct.sendgrid.net/ls/click?upn=HRESupC-2F2Czv4BOaCWWCy7my0P0qcxCbhZ31OYv50yoA22WlOQ-2By3ieUvdbKmOyw68TMVT4Kip-2BBzfOGWXJ5yIiYplmPF9KAnKIja4Zd7tU-3DLCWK_EEm8SbLgSDsaDZif-2Bv7ch8WqhKpLoKErHi4nXpwDNTuq9MPVHh3L32qrt5Ip1XnD-2FT7d2mqiVO5we8a7GYUflN8rPcFOvcPpmp7-2BHI-2FiMfMO0wZQJtLM0dmCxiZNLE1W2LBnroP7MP6w9NyH2xFZ9xER-2BYILtC7OORWk6E4iIWZD9NlPZovnnox2hXNU-2BrT4CJt7BMGDQWZi6SpY7EYshq4VQFMnD2W10PtfVsO5xns-3D To manage Coverity Scan email notifications for "tom.rini at gmail.com", click https://u15810271.ct.sendgrid.net/ls/click?upn=HRESupC-2F2Czv4BOaCWWCy7my0P0qcxCbhZ31OYv50yped04pjJnmXOsUBtKYNIXxWeIHzDeopm-2BEWQ6S6K-2FtUHv9ZTk8qZbuzkkz9sa-2BJFw4elYDyedRVZOC-2ButxjBZdouVmTGuWB6Aj6G7lm7t25-2Biv1B-2B9082pHzCCex2kqMs-3D96A1_EEm8SbLgSDsaDZif-2Bv7ch8WqhKpLoKErHi4nXpwDNTuq9MPVHh3L32qrt5Ip1XnD3WtWkNhMk8aqtcWpx18kz28O3aHVrPlQ7m76aTH42S-2FV-2BF-2BKKCm-2FUrVIBSGsRbPXbwCAkWzmG8EDOELQylE3c1UBFxOE6UpyBOxSvs1gNr-2BGyVbFqpLnYutK4cobU8DJEv-2BJRff57Ua6iETxKItKuhEjwidyxUp7lL-2FPx1HSxdY-3D ----- End forwarded message ----- -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 659 bytes Desc: not available URL: