79221cf34b
(iDefense advisories from 2005-12-05) - DCTStream Baseline Heap Overflow Vulnerability - DCTStream Progressive Heap Overflow - StreamPredictor Heap Overflow Vulnerability Patch provided by the Xpdf folks.
76 lines
2.1 KiB
Plaintext
76 lines
2.1 KiB
Plaintext
$OpenBSD: patch-pdftops_Stream_cxx,v 1.1 2005/12/06 23:23:44 mbalmer Exp $
|
|
--- pdftops/Stream.cxx.orig Mon Feb 2 23:41:09 2004
|
|
+++ pdftops/Stream.cxx Wed Dec 7 00:07:58 2005
|
|
@@ -404,18 +404,33 @@ void ImageStream::skipLine() {
|
|
|
|
StreamPredictor::StreamPredictor(Stream *strA, int predictorA,
|
|
int widthA, int nCompsA, int nBitsA) {
|
|
+ int totalBits;
|
|
+
|
|
str = strA;
|
|
predictor = predictorA;
|
|
width = widthA;
|
|
nComps = nCompsA;
|
|
nBits = nBitsA;
|
|
+ predLine = NULL;
|
|
+ ok = gFalse;
|
|
|
|
nVals = width * nComps;
|
|
+ totalBits = nVals * nBits;
|
|
+ if (totalBits == 0 ||
|
|
+ (totalBits / nBits) / nComps != width ||
|
|
+ totalBits + 7 < 0) {
|
|
+ return;
|
|
+ }
|
|
pixBytes = (nComps * nBits + 7) >> 3;
|
|
- rowBytes = ((nVals * nBits + 7) >> 3) + pixBytes;
|
|
+ rowBytes = ((totalBits + 7) >> 3) + pixBytes;
|
|
+ if (rowBytes < 0) {
|
|
+ return;
|
|
+ }
|
|
predLine = (Guchar *)gmalloc(rowBytes);
|
|
memset(predLine, 0, rowBytes);
|
|
predIdx = rowBytes;
|
|
+
|
|
+ ok = gTrue;
|
|
}
|
|
|
|
StreamPredictor::~StreamPredictor() {
|
|
@@ -991,6 +1006,10 @@ LZWStream::LZWStream(Stream *strA, int p
|
|
FilterStream(strA) {
|
|
if (predictor != 1) {
|
|
pred = new StreamPredictor(this, predictor, columns, colors, bits);
|
|
+ if (!pred->isOk()) {
|
|
+ delete pred;
|
|
+ pred = NULL;
|
|
+ }
|
|
} else {
|
|
pred = NULL;
|
|
}
|
|
@@ -2891,6 +2910,14 @@ GBool DCTStream::readBaselineSOF() {
|
|
height = read16();
|
|
width = read16();
|
|
numComps = str->getChar();
|
|
+ if (numComps <= 0 || numComps > 4) {
|
|
+ error(getPos(), "Bad number of components in DCT stream", prec);
|
|
+ return gFalse;
|
|
+ }
|
|
+ if (numComps <= 0 || numComps > 4) {
|
|
+ error(getPos(), "Bad number of components in DCT stream", prec);
|
|
+ return gFalse;
|
|
+ }
|
|
if (prec != 8) {
|
|
error(getPos(), "Bad DCT precision %d", prec);
|
|
return gFalse;
|
|
@@ -3249,6 +3276,10 @@ FlateStream::FlateStream(Stream *strA, i
|
|
FilterStream(strA) {
|
|
if (predictor != 1) {
|
|
pred = new StreamPredictor(this, predictor, columns, colors, bits);
|
|
+ if (!pred->isOk()) {
|
|
+ delete pred;
|
|
+ pred = NULL;
|
|
+ }
|
|
} else {
|
|
pred = NULL;
|
|
}
|