diff options
author | Tom N Harris <tnharris@whoopdedo.org> | 2009-01-16 01:15:47 +0100 |
---|---|---|
committer | Tom N Harris <tnharris@whoopdedo.org> | 2009-01-16 01:15:47 +0100 |
commit | ed3655c4c7f9692340f7b54054d35f24a0cc3f68 (patch) | |
tree | 1e170c6770621b7cbcb8df6da90ed950972429ff /inc/JpegMeta.php | |
parent | 1e4a1d1894f82903df44e3a3362ad2d977cb55a9 (diff) | |
download | rpg-ed3655c4c7f9692340f7b54054d35f24a0cc3f68.tar.gz rpg-ed3655c4c7f9692340f7b54054d35f24a0cc3f68.tar.bz2 |
Avoid zero-byte reads
darcs-hash:20090116001547-6942e-b5bc401c0ef4c36b1b7b8ca7ef708587953ec017.gz
Diffstat (limited to 'inc/JpegMeta.php')
-rw-r--r-- | inc/JpegMeta.php | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/inc/JpegMeta.php b/inc/JpegMeta.php index 41e6e0803..cb1d7d694 100644 --- a/inc/JpegMeta.php +++ b/inc/JpegMeta.php @@ -995,7 +995,10 @@ class JpegMeta $this->_markers[$count]['length'] = $length; if ($capture) { - $this->_markers[$count]['data'] =& fread($this->_fp, $length); + if ($length) + $this->_markers[$count]['data'] =& fread($this->_fp, $length); + else + $this->_markers[$count]['data'] = ""; } elseif (!$done) { $result = @fseek($this->_fp, $length, SEEK_CUR); @@ -1213,12 +1216,12 @@ class JpegMeta else { if ($marker == 0xDA) { // Copy until EOF while (!feof($this->_fp)) { - $data =& fread($this->_fp, 1024 * 16); + $data = fread($this->_fp, 1024 * 16); fputs($this->_fpout, $data, strlen($data)); } } else { // Copy only $length bytes - $data =& fread($this->_fp, $length); + $data = @fread($this->_fp, $length); fputs($this->_fpout, $data, $length); } } |