Merge pull request #189 from zrax/codeql

Fix CodeQL analysis for C++
This commit is contained in:
Michael Hansen
2021-04-12 10:16:07 -07:00
committed by GitHub
2 changed files with 10 additions and 13 deletions

View File

@@ -44,19 +44,16 @@ jobs:
# Autobuild attempts to build any compiled languages (C/C++, C#, or Java).
# If this step fails, then you should remove it and run the build manually (see below)
- name: Autobuild
- if: matrix.language == 'python'
name: Autobuild Python
uses: github/codeql-action/autobuild@v1
# Command-line programs to run using the OS shell.
# 📚 https://git.io/JvXDl
# ✏️ If the Autobuild fails above, remove it and uncomment the following three lines
# and modify them (or add more) to build your code if your project
# uses a compiled language
#- run: |
# make bootstrap
# make release
- if: matrix.language == 'cpp'
name: Build C++
run: |
mkdir build && cd build
cmake -DCMAKE_BUILD_TYPE=RelWithDebInfo ..
make
- name: Perform CodeQL Analysis
uses: github/codeql-action/analyze@v1

View File

@@ -58,8 +58,8 @@ const char* Pyc::OpcodeName(int opcode)
if (opcode < PYC_LAST_OPCODE)
return opcode_names[opcode];
static char badcode[10];
snprintf(badcode, 10, "<%d>", opcode);
static char badcode[16];
snprintf(badcode, sizeof(badcode), "<%d>", opcode);
return badcode;
};