diff options
| author | Christian Marangi <ansuelsmth@gmail.com> | 2023-11-12 16:15:07 +0100 |
|---|---|---|
| committer | Christian Marangi <ansuelsmth@gmail.com> | 2023-11-12 16:19:58 +0100 |
| commit | 86124a1b3a2f0b955adfbebb3f2bd2c288837aac (patch) | |
| tree | d90dac304c39befbf79bf0998d5399fbafc5e8c0 | |
| parent | fcdecb5ba46d8ec2e6c10f09d79df4f4a9300c05 (diff) | |
| download | upstream-86124a1b3a2f0b955adfbebb3f2bd2c288837aac.tar.gz upstream-86124a1b3a2f0b955adfbebb3f2bd2c288837aac.tar.bz2 upstream-86124a1b3a2f0b955adfbebb3f2bd2c288837aac.zip | |
scripts/getver.sh: prevent asking for negative rev-parse
With the case of asking an invalid version that is too big, getver.sh
might return an invalid output in the form of HEAD~-2260475641.
This is caused by BASE_REV - GET_REV using a negative number.
Prevent this by checking if BASE_REV - GET_REV actually return 0 or a
positive number and set REV variable accordingly. With the following
change, invalid revision number will result in unknown printed instead
of the invalid HEAD~-NUMBERS output.
Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>
(cherry picked from commit 9e49e0a6c4535d345084cc62c594be5cad23b911)
| -rwxr-xr-x | scripts/getver.sh | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/scripts/getver.sh b/scripts/getver.sh index 49260a2260..61906040b1 100755 --- a/scripts/getver.sh +++ b/scripts/getver.sh @@ -21,7 +21,7 @@ try_git() { r*) GET_REV="$(echo $GET_REV | tr -d 'r')" BASE_REV="$(git rev-list ${REBOOT}..HEAD 2>/dev/null | wc -l | awk '{print $1}')" - REV="$(git rev-parse HEAD~$((BASE_REV - GET_REV)))" + [ $((BASE_REV - GET_REV)) -ge 0 ] && REV="$(git rev-parse HEAD~$((BASE_REV - GET_REV)))" ;; *) BRANCH="$(git rev-parse --abbrev-ref HEAD)" |