fix for CVE-2023-28879
Resolves: rhbz#2188297
This commit is contained in:
parent
e07f2fbf61
commit
4518d41903
44
ghostscript-9.27-CVE-2023-28879.patch
Normal file
44
ghostscript-9.27-CVE-2023-28879.patch
Normal file
@ -0,0 +1,44 @@
|
|||||||
|
From 37ed5022cecd584de868933b5b60da2e995b3179 Mon Sep 17 00:00:00 2001
|
||||||
|
From: Ken Sharp <ken.sharp@artifex.com>
|
||||||
|
Date: Fri, 24 Mar 2023 13:19:57 +0000
|
||||||
|
Subject: [PATCH] Graphics library - prevent buffer overrun in (T)BCP encoding
|
||||||
|
|
||||||
|
Bug #706494 "Buffer Overflow in s_xBCPE_process"
|
||||||
|
|
||||||
|
As described in detail in the bug report, if the write buffer is filled
|
||||||
|
to one byte less than full, and we then try to write an escaped
|
||||||
|
character, we overrun the buffer because we don't check before
|
||||||
|
writing two bytes to it.
|
||||||
|
|
||||||
|
This just checks if we have two bytes before starting to write an
|
||||||
|
escaped character and exits if we don't (replacing the consumed byte
|
||||||
|
of the input).
|
||||||
|
|
||||||
|
Up for further discussion; why do we even permit a BCP encoding filter
|
||||||
|
anyway ? I think we should remove this, at least when SAFER is true.
|
||||||
|
---
|
||||||
|
base/sbcp.c | 10 +++++++++-
|
||||||
|
1 file changed, 9 insertions(+), 1 deletion(-)
|
||||||
|
|
||||||
|
diff --git a/base/sbcp.c b/base/sbcp.c
|
||||||
|
index 979ae0992..47fc233ec 100644
|
||||||
|
--- a/base/sbcp.c
|
||||||
|
+++ b/base/sbcp.c
|
||||||
|
@@ -50,6 +50,14 @@ s_xBCPE_process(stream_state * st, stream_cursor_read * pr,
|
||||||
|
byte ch = *++p;
|
||||||
|
|
||||||
|
if (ch <= 31 && escaped[ch]) {
|
||||||
|
+ /* Make sure we have space to store two characters in the write buffer,
|
||||||
|
+ * if we don't then exit without consuming the input character, we'll process
|
||||||
|
+ * that on the next time round.
|
||||||
|
+ */
|
||||||
|
+ if (pw->limit - q < 2) {
|
||||||
|
+ p--;
|
||||||
|
+ break;
|
||||||
|
+ }
|
||||||
|
if (p == rlimit) {
|
||||||
|
p--;
|
||||||
|
break;
|
||||||
|
--
|
||||||
|
2.39.2
|
||||||
|
|
@ -37,7 +37,7 @@
|
|||||||
Name: ghostscript
|
Name: ghostscript
|
||||||
Summary: Interpreter for PostScript language & PDF
|
Summary: Interpreter for PostScript language & PDF
|
||||||
Version: 9.27
|
Version: 9.27
|
||||||
Release: 8%{?dist}
|
Release: 9%{?dist}
|
||||||
|
|
||||||
License: AGPLv3+
|
License: AGPLv3+
|
||||||
|
|
||||||
@ -109,6 +109,7 @@ Patch016: ghostscript-9.27-Deal-with-different-VM-modes-during-CIDFont-loading.p
|
|||||||
Patch017: ghostscript-9.27-ESC-Page-driver-does-not-set-page-size-correctly.patch
|
Patch017: ghostscript-9.27-ESC-Page-driver-does-not-set-page-size-correctly.patch
|
||||||
Patch018: ghostscript-9.27-fix-bbox.patch
|
Patch018: ghostscript-9.27-fix-bbox.patch
|
||||||
Patch019: ghostscript-9.27-pdfwrite-Substituted-TTF-CIDFont-CID-hand.patch
|
Patch019: ghostscript-9.27-pdfwrite-Substituted-TTF-CIDFont-CID-hand.patch
|
||||||
|
Patch020: ghostscript-9.27-CVE-2023-28879.patch
|
||||||
|
|
||||||
|
|
||||||
# Downstream patches -- these should be always included when doing rebase:
|
# Downstream patches -- these should be always included when doing rebase:
|
||||||
@ -449,6 +450,10 @@ done
|
|||||||
# =============================================================================
|
# =============================================================================
|
||||||
|
|
||||||
%changelog
|
%changelog
|
||||||
|
* Fri May 05 2023 Richard Lescak <rlescak@redhat.com> - 9.27-9
|
||||||
|
- fix for CVE-2023-28879
|
||||||
|
- Resolves: rhbz#2188297
|
||||||
|
|
||||||
* Fri Mar 17 2023 Richard Lescak <rlescak@redhat.com> - 9.27-8
|
* Fri Mar 17 2023 Richard Lescak <rlescak@redhat.com> - 9.27-8
|
||||||
- fix embedding of CIDFonts
|
- fix embedding of CIDFonts
|
||||||
- Resolves: rhbz#2169890
|
- Resolves: rhbz#2169890
|
||||||
|
Loading…
Reference in New Issue
Block a user