]> granicus.if.org Git - php/commitdiff
fix imap_fetchstructure(). msgindex is not valid if we're using FT_UID;
authorChuck Hagenbuch <chagenbu@php.net>
Tue, 17 Oct 2000 16:11:26 +0000 (16:11 +0000)
committerChuck Hagenbuch <chagenbu@php.net>
Tue, 17 Oct 2000 16:11:26 +0000 (16:11 +0000)
always use Z_LVAL_PP(msgno) in the actual c-client function call.

ext/imap/php_imap.c

index d4da22188c529a25ac0c29b9b4f30b4e3cd1ddc7..a6a450998326ee6e08f374adbde068e701c5cb0f 100644 (file)
@@ -1904,16 +1904,16 @@ PHP_FUNCTION(imap_fetchstructure)
                convert_to_long_ex(flags);
        }
        object_init(return_value);
-
+    
        ind = Z_LVAL_PP(streamind);
-
+    
        imap_le_struct = (pils *) zend_list_find(ind, &ind_type);
-
+    
        if (!imap_le_struct || !IS_STREAM(ind_type)) {
                php_error(E_WARNING, "Unable to find stream pointer");
                RETURN_FALSE;
        }
-
+    
        if ((myargc == 3) && (Z_LVAL_PP(flags) & FT_UID)) {
                /*  This should be cached; if it causes an extra RTT to the
                        IMAP server, then that's the price we pay for making sure
@@ -1926,8 +1926,8 @@ PHP_FUNCTION(imap_fetchstructure)
                php_error(E_WARNING, "Bad message number");
                RETURN_FALSE;
        }
-
-       mail_fetchstructure_full(imap_le_struct->imap_stream, msgindex, &body ,myargc == 3 ? Z_LVAL_PP(flags) : NIL);
+    
+       mail_fetchstructure_full(imap_le_struct->imap_stream, Z_LVAL_PP(msgno), &body ,myargc == 3 ? Z_LVAL_PP(flags) : NIL);
        
        if (!body) {
                php_error(E_WARNING, "No body information available");