From: jangliss <jangliss@7612ce4b-ef26-0410-bec9-ea0150e637f0>
Date: Wed, 9 Nov 2005 21:35:34 +0000 (+0000)
Subject: sqimap_utf7_decode_mbx_tree doesn't issue a return, but simply changes
X-Git-Url: https://vcs.fsf.org/?a=commitdiff_plain;h=9e0f10a86a538dc55a220246072ca4b7ef1cc911;p=squirrelmail.git

sqimap_utf7_decode_mbx_tree doesn't issue a return, but simply changes
the variables by reference to the call.  This makes mbxs_tree an undefined
and unusued variable and might cause errors in php 4.4+ (especially with
objects).


git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@10309 7612ce4b-ef26-0410-bec9-ea0150e637f0
---

diff --git a/ChangeLog b/ChangeLog
index f39db56d..0a7a41f3 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -473,6 +473,8 @@ Version 1.5.1 -- CVS
   - Undefined variable in rare case in view_header.php
   - Variable by reference fix in printer_friendly_bottom.php.
   - Undefined index in addressbook backends.
+  - sqimap_utf7_decode_mbx_tree returns variables by reference, rather than a 
+    return value (#1351822)
 
 
 Version 1.5.0 - 2 February 2004
diff --git a/functions/imap_mailbox.php b/functions/imap_mailbox.php
index b0bd3f0b..a1a00fa2 100755
--- a/functions/imap_mailbox.php
+++ b/functions/imap_mailbox.php
@@ -1105,6 +1105,7 @@ function sqimap_fill_mailbox_tree($mbx_ary, $mbxs=false,$imap_stream) {
  * @since 1.5.0
  */
 function sqimap_utf7_decode_mbx_tree(&$mbx_tree) {
+
    if (strtoupper($mbx_tree->mailboxname_full) == 'INBOX')
        $mbx_tree->mailboxname_sub = _("INBOX");
    else
@@ -1112,7 +1113,7 @@ function sqimap_utf7_decode_mbx_tree(&$mbx_tree) {
    if ($mbx_tree->mbxs) {
       $iCnt = count($mbx_tree->mbxs);
       for ($i=0;$i<$iCnt;++$i) {
-          $mbxs_tree->mbxs[$i] = sqimap_utf7_decode_mbx_tree($mbx_tree->mbxs[$i]);
+            sqimap_utf7_decode_mbx_tree($mbx_tree->mbxs[$i]);
       }
    }
 }
@@ -1252,4 +1253,4 @@ function sqimap_mailbox_is_noinferiors($oImapStream,$sImapFolder,&$oBoxes) {
     return false;
 }
 
-?>
\ No newline at end of file
+?>