0ct0pu5/squirrelmail
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Due to changed behaviour of fread (see php bug 24033) we cannot use large

Browse source 
buffersizes anymore because we cannot trust fread if the requested buffer is
actually returned.
blocking socket connections do not work anymore like they supposed to work
:(
This commit is contained in:
stekkel 2003-07-24 21:23:18 +00:00
parent e83198d1f1
commit e375cf726b
1 changed files with 2 additions and 12 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
functions/imap_general.php
View file

@ -187,7 +187,8 @@ function sqimap_fread($imap_stream,$iSize,$filter=false,
if (!$filter || !$outputstream) {
$iBufferSize = $iSize;
} else {
$iBufferSize = 62400; // multiple of 78 in case of base64 decoding.
// see php bug 24033. They changed fread behaviour %$^&$%
$iBufferSize = 780; // multiple of 78 in case of base64 decoding.
}
$iRet = $iSize - $iBufferSize;
$iRetrieved = 0;
@ -201,17 +202,6 @@ function sqimap_fread($imap_stream,$iSize,$filter=false,
break;
}
$iRetrieved += $iBufferSize;
// if the returned lines are split, do not end with \n
// then we have a problem and need to adjust (happened with uw)
if ($bBufferSizeAdapted && substr($sRead,-1) !== "\n") {
// use fgets because it stops at \n.
// we can do it because it's for correction
$sRead .= fgets($imap_stream,$iBufferSize);
$iRetrieved += strlen($sRead);
if ($iRetrieved == $iSize) {
$bFinished = true;
}
}
if ($filter) {
// in case line-endings do not appear at position 78 we adapt the buffersize so we can base64 decode on the fly
if (!$bBufferSizeAdapted) {