Re: possible imap.el bug

[Simon Josefsson <jas@extundo.com>]

"Ted Zlatanov" <tzz@lifelogs.com> writes:

> On Fri, 15 Oct 2004, jas@extundo.com wrote:
>
>> "Ted Zlatanov" <tzz@lifelogs.com> writes:
>
>>> 10845 UID COPY 32361 "soccer".
>>> 10845 OK UID COPY completed.
>> 
>> Could you get the CAPABILITY line from that server too?  First lines
>> when connecting to the server.
>
> * CAPABILITY IMAP4 IMAP4REV1 NAMESPACE IDLE SCAN SORT MAILBOX-REFERRALS LOGIN-REFERRALS AUTH=LOGIN THREAD=ORDEREDSUBJECT
>
>> The imap.el code check for the UIDPLUS capability (RFC 2359) and uses
>> some special code if present, and it is the code that crashes.  If the
>> server really supported UIDPLUS, the UID COPY response should be
>> something like:
>> 
>> A003 OK [COPYUID 4711] Done
>> 
>> If the capability line includes UIDPLUS, then this is a server bug.
>
> So maybe imap.el tried UIDPLUS even though it shouldn't...

Weird.  The code says:

(defun imap-message-copyuid-1 (mailbox)
  (if (imap-capability 'UIDPLUS)
      (list (nth 0 (imap-mailbox-get-1 'copyuid mailbox))
	    (string-to-number (nth 2 (imap-mailbox-get-1 'copyuid mailbox))))
    (let ((old-mailbox imap-current-mailbox)
	  (state imap-state)
	  (imap-message-data (make-vector 2 0)))
      (when (imap-mailbox-examine-1 mailbox)
	(prog1
	    (and (imap-fetch "*" "UID")
		 (list (imap-mailbox-get-1 'uidvalidity mailbox)
		       (apply 'max (imap-message-map
				    (lambda (uid prop) uid) 'UID))))
	  (if old-mailbox
	      (imap-mailbox-select old-mailbox (eq state 'examine))
	    (imap-mailbox-unselect)))))))

Could you (load "imap.el") to get a more detailed backtrace, perhaps?
I thought it was the string-to-number in the UIDPLUS part of the big
if that crashed, but maybe it is some other call, hidden inside the
rest of the function.

Or the UIDPLUS capability has leaked between buffers.