From dc92574c10f3e2516ec6445b88c5d584f40df4e5 Mon Sep 17 00:00:00 2001 From: Jay Berkenbilt Date: Mon, 4 Jan 2021 11:55:28 -0500 Subject: Fix some pipelines to be safe if downstream write fails (fuzz issue 28262) --- libqpdf/Pl_ASCII85Decoder.cc | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) (limited to 'libqpdf/Pl_ASCII85Decoder.cc') diff --git a/libqpdf/Pl_ASCII85Decoder.cc b/libqpdf/Pl_ASCII85Decoder.cc index b8df3e87..9d9f6704 100644 --- a/libqpdf/Pl_ASCII85Decoder.cc +++ b/libqpdf/Pl_ASCII85Decoder.cc @@ -119,10 +119,13 @@ Pl_ASCII85Decoder::flush() QTC::TC("libtests", "Pl_ASCII85Decoder partial flush", (this->pos == 5) ? 0 : 1); - getNext()->write(outbuf, this->pos - 1); - + // Reset before calling getNext()->write in case that throws an + // exception. + auto t = this->pos - 1; this->pos = 0; memset(this->inbuf, 117, 5); + + getNext()->write(outbuf, t); } void -- cgit v1.2.3-54-g00ecf