Really fix UNKNOWN-CTE handling

Aleksander Machniak

2014-10-23 a23c233db6d34aa8fab96d5d9d708a373bf4fa0f

Really fix UNKNOWN-CTE handling

 program/lib/Roundcube/rcube_imap_generic.php

@@ -2531,6 +2531,7 @@
        }

        $initiated = false;
        $binary    = true;

        do {
            if (!$initiated) {
@@ -2552,16 +2553,16 @@
                }

                // Use BINARY extension when possible (and safe)
                $binary     = !$binary_err && $mode && preg_match('/^[0-9.]+$/', $part) && $this->hasCapability('BINARY');
                $binary     = $binary && $mode && preg_match('/^[0-9.]+$/', $part) && $this->hasCapability('BINARY');
                $fetch_mode = $binary ? 'BINARY' : 'BODY';
                $partial    = $max_bytes ? sprintf('<0.%d>', $max_bytes) : '';

                // format request
                $key        = $this->nextTag();
                $request    = $key . ($is_uid ? ' UID' : '') . " FETCH $id ($fetch_mode.PEEK[$part]$partial)";
                $result     = false;
                $found      = false;
                $initiated  = true;
                $key       = $this->nextTag();
                $request   = $key . ($is_uid ? ' UID' : '') . " FETCH $id ($fetch_mode.PEEK[$part]$partial)";
                $result    = false;
                $found     = false;
                $initiated = true;

                // send request
                if (!$this->putLine($request)) {
@@ -2577,16 +2578,16 @@

            $line = trim($this->readLine(1024));

            // handle UNKNOWN-CTE response - RFC 3516, try standard BODY request instead of BINARY
            if ($binary && preg_match('/^' . $key . ' NO \[UNKNOWN-CTE\]/i', $line)) {
                $initiated = false;
                continue;
            }

            if (!$line) {
                break;
            }

            // handle UNKNOWN-CTE response - RFC 3516, try again with standard BODY request
            if ($binary && !$found && preg_match('/^' . $key . ' NO \[UNKNOWN-CTE\]/i', $line)) {
                $binary = $initiated = false;
                continue;
            }

            // skip irrelevant untagged responses (we have a result already)
            if ($found || !preg_match('/^\* ([0-9]+) FETCH (.*)$/', $line, $m)) {
                continue;