Skip to content

Commit a81c8bc

Browse files
authored
Merge pull request #79 from qualcomm-linux/mynameistechno-patch-1
Prevent code injection in workflow
2 parents 7c9eeae + a701f38 commit a81c8bc

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

.github/workflows/test.yml

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,14 +20,16 @@ jobs:
2020
outputs:
2121
jobmatrix: ${{ steps.listjobs.outputs.jobmatrix }}
2222
steps:
23+
- env:
24+
BUILD_URL: ${{ inputs.url }}
2325
- name: Clone repository
2426
uses: actions/checkout@v4
2527
with:
2628
fetch-depth: 0
2729
- name: Print trigger
2830
run: |
2931
echo "Triggered by ${{ github.event_name }}"
30-
echo "Build URL: ${{ inputs.url }}"
32+
echo "Build URL: $BUILD_URL"
3133
- name: "List jobs"
3234
id: listjobs
3335
run: |
@@ -56,7 +58,7 @@ jobs:
5658
FIND_PATH="${TARGET#*/}"
5759
DEVICE_TYPE_PATH="${FIND_PATH%/*}"
5860
DEVICE_TYPE="${DEVICE_TYPE_PATH#*/}"
59-
BUILD_DOWNLOAD_URL="${{inputs.url}}"
61+
BUILD_DOWNLOAD_URL="$BUILD_URL"
6062
sed -i "s|{{DEVICE_TYPE}}|${DEVICE_TYPE}|g" "${{ matrix.target }}"
6163
sed -i "s|{{GITHUB_SHA}}|${GITHUB_SHA}|g" "${{ matrix.target }}"
6264
sed -i "s|{{BUILD_DOWNLOAD_URL}}|${BUILD_DOWNLOAD_URL}|g" "${{ matrix.target }}"

0 commit comments

Comments
 (0)