From c8846d9819bf48d8a7b864d4d4790c64f297934c Mon Sep 17 00:00:00 2001 From: msslulu <1484036491@qq.com> Date: Tue, 7 Jul 2026 04:36:44 -0700 Subject: [PATCH 1/2] Add CodeQL analysis workflow configuration This workflow file configures CodeQL analysis for the repository, specifying triggers, jobs, and steps for analyzing various programming languages. --- .github/workflows/codeql.yml | 101 +++++++++++++++++++++++++++++++++++ 1 file changed, 101 insertions(+) create mode 100644 .github/workflows/codeql.yml diff --git a/.github/workflows/codeql.yml b/.github/workflows/codeql.yml new file mode 100644 index 00000000..15037685 --- /dev/null +++ b/.github/workflows/codeql.yml @@ -0,0 +1,101 @@ +# For most projects, this workflow file will not need changing; you simply need +# to commit it to your repository. +# +# You may wish to alter this file to override the set of languages analyzed, +# or to provide custom queries or build logic. +# +# ******** NOTE ******** +# We have attempted to detect the languages in your repository. Please check +# the `language` matrix defined below to confirm you have the correct set of +# supported CodeQL languages. +# +name: "CodeQL Advanced" + +on: + push: + branches: [ "develop" ] + pull_request: + branches: [ "develop" ] + schedule: + - cron: '24 15 * * 1' + +jobs: + analyze: + name: Analyze (${{ matrix.language }}) + # Runner size impacts CodeQL analysis time. To learn more, please see: + # - https://gh.io/recommended-hardware-resources-for-running-codeql + # - https://gh.io/supported-runners-and-hardware-resources + # - https://gh.io/using-larger-runners (GitHub.com only) + # Consider using larger runners or machines with greater resources for possible analysis time improvements. + runs-on: ${{ (matrix.language == 'swift' && 'macos-latest') || 'ubuntu-latest' }} + permissions: + # required for all workflows + security-events: write + + # required to fetch internal or private CodeQL packs + packages: read + + # only required for workflows in private repositories + actions: read + contents: read + + strategy: + fail-fast: false + matrix: + include: + - language: actions + build-mode: none + - language: java-kotlin + build-mode: none # This mode only analyzes Java. Set this to 'autobuild' or 'manual' to analyze Kotlin too. + # CodeQL supports the following values keywords for 'language': 'actions', 'c-cpp', 'csharp', 'go', 'java-kotlin', 'javascript-typescript', 'python', 'ruby', 'rust', 'swift' + # Use `c-cpp` to analyze code written in C, C++ or both + # Use 'java-kotlin' to analyze code written in Java, Kotlin or both + # Use 'javascript-typescript' to analyze code written in JavaScript, TypeScript or both + # To learn more about changing the languages that are analyzed or customizing the build mode for your analysis, + # see https://docs.github.com/en/code-security/code-scanning/creating-an-advanced-setup-for-code-scanning/customizing-your-advanced-setup-for-code-scanning. + # If you are analyzing a compiled language, you can modify the 'build-mode' for that language to customize how + # your codebase is analyzed, see https://docs.github.com/en/code-security/code-scanning/creating-an-advanced-setup-for-code-scanning/codeql-code-scanning-for-compiled-languages + steps: + - name: Checkout repository + uses: actions/checkout@v4 + + # Add any setup steps before running the `github/codeql-action/init` action. + # This includes steps like installing compilers or runtimes (`actions/setup-node` + # or others). This is typically only required for manual builds. + # - name: Setup runtime (example) + # uses: actions/setup-example@v1 + + # Initializes the CodeQL tools for scanning. + - name: Initialize CodeQL + uses: github/codeql-action/init@v4 + with: + languages: ${{ matrix.language }} + build-mode: ${{ matrix.build-mode }} + # If you wish to specify custom queries, you can do so here or in a config file. + # By default, queries listed here will override any specified in a config file. + # Prefix the list here with "+" to use these queries and those in the config file. + + # For more details on CodeQL's query packs, refer to: https://docs.github.com/en/code-security/code-scanning/automatically-scanning-your-code-for-vulnerabilities-and-errors/configuring-code-scanning#using-queries-in-ql-packs + # queries: security-extended,security-and-quality + + # If the analyze step fails for one of the languages you are analyzing with + # "We were unable to automatically build your code", modify the matrix above + # to set the build mode to "manual" for that language. Then modify this step + # to build your code. + # ℹ️ Command-line programs to run using the OS shell. + # 📚 See https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#jobsjob_idstepsrun + - name: Run manual build steps + if: matrix.build-mode == 'manual' + shell: bash + run: | + echo 'If you are using a "manual" build mode for one or more of the' \ + 'languages you are analyzing, replace this with the commands to build' \ + 'your code, for example:' + echo ' make bootstrap' + echo ' make release' + exit 1 + + - name: Perform CodeQL Analysis + uses: github/codeql-action/analyze@v4 + with: + category: "/language:${{matrix.language}}" From 1d81338b3875b036da42f803ee65bc9e8ffb4fa8 Mon Sep 17 00:00:00 2001 From: msslulu <1484036491@qq.com> Date: Wed, 8 Jul 2026 02:30:54 -0700 Subject: [PATCH 2/2] fix:codeQL --- .github/workflows/codeql-config.yml | 4 + .github/workflows/codeql.yml | 3 +- .../tinyengine/it/test/VulnerableTest.java | 88 +++++++++++++++++++ 3 files changed, 94 insertions(+), 1 deletion(-) create mode 100644 .github/workflows/codeql-config.yml create mode 100644 base/src/main/java/com/tinyengine/it/test/VulnerableTest.java diff --git a/.github/workflows/codeql-config.yml b/.github/workflows/codeql-config.yml new file mode 100644 index 00000000..41e6c9ff --- /dev/null +++ b/.github/workflows/codeql-config.yml @@ -0,0 +1,4 @@ +paths-ignore: + - '**/target/' # Maven 编译输出 + - '**/src/test/' # 单元测试代码（测试断言常被误报） + - '**/generated-sources/' # 自动生成的代码（如 MapStruct、QueryDSL） \ No newline at end of file diff --git a/.github/workflows/codeql.yml b/.github/workflows/codeql.yml index 15037685..d29f6ca6 100644 --- a/.github/workflows/codeql.yml +++ b/.github/workflows/codeql.yml @@ -46,7 +46,7 @@ jobs: - language: actions build-mode: none - language: java-kotlin - build-mode: none # This mode only analyzes Java. Set this to 'autobuild' or 'manual' to analyze Kotlin too. + build-mode: autobuild # This mode only analyzes Java. Set this to 'autobuild' or 'manual' to analyze Kotlin too. # CodeQL supports the following values keywords for 'language': 'actions', 'c-cpp', 'csharp', 'go', 'java-kotlin', 'javascript-typescript', 'python', 'ruby', 'rust', 'swift' # Use `c-cpp` to analyze code written in C, C++ or both # Use 'java-kotlin' to analyze code written in Java, Kotlin or both @@ -71,6 +71,7 @@ jobs: with: languages: ${{ matrix.language }} build-mode: ${{ matrix.build-mode }} + config-file: ./.github/codeql/codeql-config.yml # <--- 加上这一行 # If you wish to specify custom queries, you can do so here or in a config file. # By default, queries listed here will override any specified in a config file. # Prefix the list here with "+" to use these queries and those in the config file. diff --git a/base/src/main/java/com/tinyengine/it/test/VulnerableTest.java b/base/src/main/java/com/tinyengine/it/test/VulnerableTest.java new file mode 100644 index 00000000..6fb5b237 --- /dev/null +++ b/base/src/main/java/com/tinyengine/it/test/VulnerableTest.java @@ -0,0 +1,88 @@ +package com.tinyengine.it.test; + +import jakarta.servlet.http.HttpServletRequest; +import org.springframework.web.bind.annotation.GetMapping; +import org.springframework.web.bind.annotation.RequestParam; +import org.springframework.web.bind.annotation.RestController; + +import java.io.File; +import java.io.FileReader; +import java.io.IOException; +import java.net.HttpURLConnection; +import java.net.URL; +import java.sql.*; + +// 这是一个故意包含漏洞的测试类，用于验证CodeQL扫描功能 +@RestController +public class VulnerableTest { + // 1. SQL 注入漏洞 (CWE-89) + // 漏洞点：直接将用户输入拼接到SQL语句中 + @GetMapping("/sql-injection") + public String sqlInjection(@RequestParam String productId, HttpServletRequest request) { + try { + Connection conn = DriverManager.getConnection("jdbc:mysql://localhost/test", "user", "pass"); + // 危险：直接拼接用户输入 + String query = "SELECT * FROM products WHERE id = " + productId; // $source + Statement st = conn.createStatement(); + ResultSet rs = st.executeQuery(query); // $sink + // ... 处理结果 + return "Query executed"; + } catch (SQLException e) { + return "Error: " + e.getMessage(); + } + } + + // 2. 命令注入漏洞 (CWE-78) + // 漏洞点：用户输入被直接用于系统命令执行 + @GetMapping("/command-injection") + public String commandInjection(@RequestParam String filename) { + try { + // 危险：用户输入直接拼接到命令中 + Runtime runtime = Runtime.getRuntime(); + Process process = runtime.exec("ls -l " + filename); // $source -> $sink + // ... 处理结果 + return "Command executed"; + } catch (IOException e) { + return "Error: " + e.getMessage(); + } + } + + // 3. SSRF 服务端请求伪造漏洞 (CWE-918) + // 漏洞点：用户控制的URL被用于发起网络请求 + @GetMapping("/ssrf") + public String ssrf(@RequestParam String urlParam) { + try { + // 危险：用户输入直接构造URL并发起连接 + URL url = new URL(urlParam); // $source + HttpURLConnection conn = (HttpURLConnection) url.openConnection(); // $sink + // ... 处理响应 + return "Request sent"; + } catch (IOException e) { + return "Error: " + e.getMessage(); + } + } + + // 4. 硬编码密码/密钥 (CWE-798) + // 漏洞点：敏感信息直接写在代码中 + public void hardcodedCredential() { + String password = "admin123"; // $hardcoded + String apiKey = "sk-1234567890abcdef"; // $hardcoded + // ... + } + + // 5. 路径遍历漏洞 (CWE-22) + // 漏洞点：用户输入直接用于构造文件路径 + @GetMapping("/file-read") + public String fileRead(@RequestParam String filename) { + try { + // 危险：用户输入直接用于文件读取 + File file = new File("/var/data/" + filename); // $source + FileReader fr = new FileReader(file); // $sink + // ... 读取文件内容 + return "File read"; + } catch (IOException e) { + return "Error: " + e.getMessage(); + } + } + +}