Leonardo Sandoval | 9dfdd1b | 2020-08-06 17:08:11 -0500 | [diff] [blame] | 1 | #!/usr/bin/env bash |
Fathi Boudra | 422bf77 | 2019-12-02 11:10:16 +0200 | [diff] [blame] | 2 | # |
Govindraj Raja | 4db3c00 | 2025-04-10 17:23:19 -0500 | [diff] [blame] | 3 | # Copyright (c) 2019-2025 Arm Limited. All rights reserved. |
Fathi Boudra | 422bf77 | 2019-12-02 11:10:16 +0200 | [diff] [blame] | 4 | # |
| 5 | # SPDX-License-Identifier: BSD-3-Clause |
| 6 | # |
| 7 | |
| 8 | echo '----------------------------------------------' |
| 9 | echo '-- Running static checks on the source code --' |
| 10 | echo '----------------------------------------------' |
| 11 | |
| 12 | # Find the absolute path of the scripts' top directory |
| 13 | |
| 14 | cd "$(dirname "$0")/../.." |
| 15 | export CI_ROOT=$(pwd) |
| 16 | cd - |
| 17 | |
Chris Kay | ba047fe | 2025-07-31 16:13:20 +0100 | [diff] [blame] | 18 | git fetch --unshallow --update-shallow origin |
| 19 | git fetch --unshallow --update-shallow origin ${GERRIT_BRANCH} ${GERRIT_REFSPEC} |
Paul Sokolovsky | 74cf0f1 | 2024-06-18 23:21:07 +0300 | [diff] [blame] | 20 | |
Chris Kay | ba047fe | 2025-07-31 16:13:20 +0100 | [diff] [blame] | 21 | export merge_base=$(git merge-base \ |
| 22 | $(head -n1 .git/FETCH_HEAD | cut -f1) \ |
| 23 | $(tail -n1 .git/FETCH_HEAD | cut -f1)) |
Fathi Boudra | 422bf77 | 2019-12-02 11:10:16 +0200 | [diff] [blame] | 24 | |
| 25 | export LOG_TEST_FILENAME=$(pwd)/static-checks.log |
| 26 | |
| 27 | echo |
| 28 | echo "###### Static checks ######" |
| 29 | echo |
| 30 | |
| 31 | echo "###### Static checks ######" > "$LOG_TEST_FILENAME" |
| 32 | echo >> "$LOG_TEST_FILENAME" |
| 33 | |
Paul Sokolovsky | 74cf0f1 | 2024-06-18 23:21:07 +0300 | [diff] [blame] | 34 | echo "Patch series being checked:" >> "$LOG_TEST_FILENAME" |
Harrison Mutai | 789decf | 2025-07-16 08:26:58 +0000 | [diff] [blame] | 35 | git log --oneline ${merge_base}..HEAD >> "$LOG_TEST_FILENAME" |
Paul Sokolovsky | 74cf0f1 | 2024-06-18 23:21:07 +0300 | [diff] [blame] | 36 | echo >> "$LOG_TEST_FILENAME" |
| 37 | echo "Base branch reference commit:" >> "$LOG_TEST_FILENAME" |
Harrison Mutai | 789decf | 2025-07-16 08:26:58 +0000 | [diff] [blame] | 38 | git log --oneline -1 ${merge_base} >> "$LOG_TEST_FILENAME" |
| 39 | |
Paul Sokolovsky | 74cf0f1 | 2024-06-18 23:21:07 +0300 | [diff] [blame] | 40 | |
| 41 | echo >> "$LOG_TEST_FILENAME" |
| 42 | |
Fathi Boudra | 422bf77 | 2019-12-02 11:10:16 +0200 | [diff] [blame] | 43 | # Reset error counters |
| 44 | |
| 45 | ERROR_COUNT=0 |
| 46 | WARNING_COUNT=0 |
| 47 | |
| 48 | # Ensure all the files contain a copyright |
| 49 | |
| 50 | echo 'Checking copyright in source files...' |
| 51 | echo |
| 52 | "$CI_ROOT"/script/static-checks/static-checks-check-copyright.sh . |
| 53 | if [ "$?" != 0 ]; then |
| 54 | echo "Copyright test: FAILURE" |
| 55 | ((ERROR_COUNT++)) |
| 56 | else |
| 57 | echo "Copyright test: PASS" |
| 58 | fi |
| 59 | echo |
| 60 | |
| 61 | # Check alphabetic order of headers included. |
| 62 | |
| 63 | if [ "$IS_CONTINUOUS_INTEGRATION" == 1 ]; then |
| 64 | "$CI_ROOT"/script/static-checks/static-checks-include-order.sh . patch |
| 65 | else |
| 66 | "$CI_ROOT"/script/static-checks/static-checks-include-order.sh . |
| 67 | fi |
| 68 | if [ "$?" != 0 ]; then |
| 69 | echo "Include order test: FAILURE" |
| 70 | ((WARNING_COUNT++)) |
| 71 | else |
| 72 | echo "Include order test: PASS" |
| 73 | fi |
| 74 | echo |
| 75 | |
Govindraj Raja | 4db3c00 | 2025-04-10 17:23:19 -0500 | [diff] [blame] | 76 | # Check ascending order of CPU ERRATUM and CVE added. |
| 77 | |
| 78 | if [ "$IS_CONTINUOUS_INTEGRATION" == 1 ]; then |
| 79 | "$CI_ROOT"/script/static-checks/static-checks-cpu-erratum-order.sh . patch |
| 80 | else |
| 81 | "$CI_ROOT"/script/static-checks/static-checks-cpu-erratum-order.sh . |
| 82 | fi |
| 83 | if [ "$?" != 0 ]; then |
| 84 | echo "CPU Errata, CVE in-order test: FAILURE" |
| 85 | ((ERROR_COUNT++)) |
| 86 | else |
| 87 | echo "CPU Errata, CVE in-order test: PASS" |
| 88 | fi |
| 89 | echo |
| 90 | |
Fathi Boudra | 422bf77 | 2019-12-02 11:10:16 +0200 | [diff] [blame] | 91 | # Check line endings |
| 92 | |
Leonardo Sandoval | 239e8ac | 2020-07-27 15:14:59 -0500 | [diff] [blame] | 93 | if [ "$IS_CONTINUOUS_INTEGRATION" == 1 ]; then |
| 94 | "$CI_ROOT"/script/static-checks/static-checks-coding-style-line-endings.sh . patch |
| 95 | else |
| 96 | "$CI_ROOT"/script/static-checks/static-checks-coding-style-line-endings.sh |
| 97 | fi |
| 98 | |
Fathi Boudra | 422bf77 | 2019-12-02 11:10:16 +0200 | [diff] [blame] | 99 | if [ "$?" != 0 ]; then |
| 100 | echo "Line ending test: FAILURE" |
| 101 | ((ERROR_COUNT++)) |
| 102 | else |
| 103 | echo "Line ending test: PASS" |
| 104 | fi |
| 105 | echo |
| 106 | |
| 107 | # Check coding style |
| 108 | |
| 109 | echo 'Checking coding style compliance...' |
| 110 | echo |
| 111 | if [ "$IS_CONTINUOUS_INTEGRATION" == 1 ]; then |
| 112 | "$CI_ROOT"/script/static-checks/static-checks-coding-style.sh |
| 113 | else |
| 114 | "$CI_ROOT"/script/static-checks/static-checks-coding-style-entire-src-tree.sh |
| 115 | fi |
| 116 | if [ "$?" != 0 ]; then |
| 117 | echo "Coding style test: FAILURE" |
| 118 | ((ERROR_COUNT++)) |
| 119 | else |
| 120 | echo "Coding style test: PASS" |
| 121 | fi |
| 122 | echo |
| 123 | |
Zelalem | 219df41 | 2020-05-17 19:21:20 -0500 | [diff] [blame] | 124 | # Check for any Banned API usage |
| 125 | |
| 126 | echo 'Checking Banned API usage...' |
| 127 | echo |
| 128 | if [ "$IS_CONTINUOUS_INTEGRATION" == 1 ]; then |
| 129 | "$CI_ROOT"/script/static-checks/static-checks-banned-apis.sh . patch |
| 130 | else |
| 131 | "$CI_ROOT"/script/static-checks/static-checks-banned-apis.sh |
| 132 | fi |
| 133 | if [ "$?" != 0 ]; then |
| 134 | echo "Banned API check: FAILURE" |
| 135 | ((ERROR_COUNT++)) |
| 136 | else |
| 137 | echo "Banned API check: PASS" |
| 138 | fi |
| 139 | echo |
| 140 | |
Jayanth Dodderi Chidanand | 5132cb1 | 2021-08-09 17:54:47 +0100 | [diff] [blame] | 141 | # Check to ensure newly added source files are detected for Coverity Scan analysis |
| 142 | |
Jayanth Dodderi Chidanand | 253b339 | 2021-09-10 12:40:00 +0100 | [diff] [blame] | 143 | # Check to be executed only on trusted-firmware repository. |
Manish V Badarkhe | 8b846f4 | 2021-10-13 15:43:58 +0100 | [diff] [blame] | 144 | if [ "$REPO_UNDER_TEST" = "trusted-firmware" ] || [ "$REPO_UNDER_TEST" = "trusted-firmware-a" ]; then |
Jayanth Dodderi Chidanand | 253b339 | 2021-09-10 12:40:00 +0100 | [diff] [blame] | 145 | echo 'Checking whether the newly added source files are detected for Coverity Scan analysis...' |
| 146 | echo |
| 147 | "$CI_ROOT"/script/static-checks/static-checks-detect-newly-added-files.sh |
| 148 | if [ "$?" != 0 ]; then |
| 149 | echo "Files Detection check: FAILURE" |
| 150 | ((ERROR_COUNT++)) |
| 151 | else |
| 152 | echo "Files Detection check: PASS" |
| 153 | fi |
| 154 | echo |
Jayanth Dodderi Chidanand | 5132cb1 | 2021-08-09 17:54:47 +0100 | [diff] [blame] | 155 | fi |
Zelalem | 219df41 | 2020-05-17 19:21:20 -0500 | [diff] [blame] | 156 | |
Fathi Boudra | 422bf77 | 2019-12-02 11:10:16 +0200 | [diff] [blame] | 157 | # Check error count |
| 158 | |
| 159 | if [ "$ERROR_COUNT" != 0 ] || [ "$WARNING_COUNT" != 0 ]; then |
| 160 | echo "Some static checks have failed." |
| 161 | fi |
| 162 | |
| 163 | if [ "$ERROR_COUNT" != 0 ]; then |
| 164 | exit 1 |
| 165 | fi |
| 166 | |
| 167 | exit 0 |