From: ThorstenB <brehmt@gmail.com>
Date: Sat, 10 Nov 2012 09:12:01 +0000 (+0100)
Subject: Proper fix for gzfilebuf choking on some files.
X-Git-Url: https://git.mxchange.org/?a=commitdiff_plain;h=0928bca531938db3f9ea795530dfef21665c5c98;p=simgear.git

Proper fix for gzfilebuf choking on some files.
When reading a new block of data, and the first byte of the new block is
0xff (_signed_ char -1), then this must be converted to _integer_ 0xff
(+255), not -1 - which would indicate an error condition (EOF==-1). All
valid _data_ character must be returned as non-negative, see
streambuf::underflow spec, or compare with
http://www.opensource.apple.com/source/zlib/zlib-12/zlib/contrib/iostream/zfstream.cpp
or
http://www.raspberryginger.com/jbailey/minix/html/zfstream_8cpp-source.html
---

diff --git a/simgear/misc/zfstream.cxx b/simgear/misc/zfstream.cxx
index 38a16ff2..eae24f78 100644
--- a/simgear/misc/zfstream.cxx
+++ b/simgear/misc/zfstream.cxx
@@ -270,7 +270,7 @@ gzfilebuf::underflow()
     }
     else
     {
-        return fillbuf() == EOF ? traits_type::eof() : int_type(*gptr());
+        return fillbuf() == EOF ? traits_type::eof() : (unsigned char) (*gptr());
     }
 }