[dev] BUG horde/lib/mime/viewer/smime.php

Mike Cochrane mike@graftonhall.co.nz
Thu, 8 Aug 2002 15:16:40 +0100


Okay... so this is partly my IMAP server.. :-( 

it doesn't return the structure right.. but it would be nice if this failed a
bit more gracefully... 


- Mike :-)

Quoting Mike Cochrane <mike@graftonhall.co.nz>:

> 
> there's an infinite recursion bug in smime.php caused my (i suspect) a bug in
> MIME_Part.
> 
> Here's the offending code (with some debuging code):
> 
>  function resolveViewer()
>     {
>         static $depth;
>         if ($depth > 5) return false;
>         Horde::logMessage('SMIME::resolveViwer() depth:' . $depth++,
> __FILE__,
> __LINE__, LOG_DEBUG);
> 
>         $viewer = null;
>         if (empty($this->viewer)) {
> 
> //           
> var_dump($this->mime_part->getContentTypeParameter('protocol'));
> 
>             $viewer = MIME_Viewer::factory($this->mime_part,
> $this->mime_part->getContentTypeParameter('protocol'));
>             if (empty($viewer) || $viewer === false || get_class($viewer) ==
> 'mime_viewer_default') {
>                 $viewer = MIME_Viewer::factory($this->mime_part,
> $this->mime_part->getPrimaryType() . '/*');
>                 if (empty($viewer) || $viewer === false) {
>                     return false;
>                 }
>             }
>             $this->viewer = $viewer;
>         }
> 
>         return true;
>     }
> 
> The Problem:
> the $this->mime_part->getContentTypeParameter('protocol') call always returns
> false as the 'protocol' parameter is never set in MIME_Part. 
> this causes MIME_Viewer::factory() to return another MIME_Viewer_smime and we
> recurse indefinatly. I have no idea what is different about my system here
> that
> noone else has notices this. PHP just recurses until if FAULTs here... Win32.
> 
> The 3 lines at the top of the code are my debugging code and to stop it
> recusing
> too far.
> 
> And idea's Michael?
> 
> Thanks
> 
> - Mike :-)
> 
> --
> 
> -------------------------------------------------------------------------
> This mail sent from Mike's CVS HEAD install of IMP: http://horde.org/imp/
> 
> 
> -- 
> Horde developers mailing list
> Frequently Asked Questions: http://horde.org/faq/
> To unsubscribe, mail: dev-unsubscribe@lists.horde.org


--

-------------------------------------------------------------------------
This mail sent from Mike's CVS HEAD install of IMP: http://horde.org/imp/