-
Notifications
You must be signed in to change notification settings - Fork 506
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
gawk: backport Debian patch to fix CVE-2023-4156
Upstream-Status: Backport [https://git.launchpad.net/ubuntu/+source/gawk/tree/debian/patches?h=ubuntu/jammy-security & https://git.savannah.gnu.org/gitweb/?p=gawk.git;a=commitdiff;h=e709eb829448ce040087a3fc5481db6bfcaae212] Signed-off-by: Vijay Anusuri <[email protected]> Signed-off-by: Steve Sakoman <[email protected]>
- Loading branch information
1 parent
a1256b8
commit 0547b60
Showing
2 changed files
with
29 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,28 @@ | ||
From e709eb829448ce040087a3fc5481db6bfcaae212 Mon Sep 17 00:00:00 2001 | ||
From: "Arnold D. Robbins" <[email protected]> | ||
Date: Wed, 3 Aug 2022 13:00:54 +0300 | ||
Subject: [PATCH] Smal bug fix in builtin.c. | ||
|
||
Upstream-Status: Backport [import from ubuntu https://git.launchpad.net/ubuntu/+source/gawk/tree/debian/patches/CVE-2023-4156.patch?h=ubuntu/jammy-security | ||
Upstream commit https://git.savannah.gnu.org/gitweb/?p=gawk.git;a=commitdiff;h=e709eb829448ce040087a3fc5481db6bfcaae212] | ||
CVE: CVE-2023-4156 | ||
Signed-off-by: Vijay Anusuri <[email protected]> | ||
--- | ||
ChangeLog | 6 ++++++ | ||
builtin.c | 5 ++++- | ||
2 files changed, 10 insertions(+), 1 deletion(-) | ||
|
||
--- gawk-5.1.0.orig/builtin.c | ||
+++ gawk-5.1.0/builtin.c | ||
@@ -957,7 +957,10 @@ check_pos: | ||
s1++; | ||
n0--; | ||
} | ||
- if (val >= num_args) { | ||
+ // val could be less than zero if someone provides a field width | ||
+ // so large that it causes integer overflow. Mainly fuzzers do this, | ||
+ // but let's try to be good anyway. | ||
+ if (val < 0 || val >= num_args) { | ||
toofew = true; | ||
break; | ||
} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters