bug#20956: [PATCH] Fix imap message size parsing

Discussion:

Nikolaus Rath

2015-07-02 02:45:05 UTC

Package: gnus
Tags: patch

IMAP servers may sent the data items in a FETCH response in arbitrary
order. However, currently Gnus assumes that the UID item is sent before
the RFC822.SIZE item. If this assumption is wrong, Gnus is unable to
determine the message size.

The attached patch fixes this issue.

Best,
-Nikolaus

--
GPG encrypted emails preferred. Key id: 0xD113FCAC3C4E599F
Fingerprint: ED31 791B 2C5C 1613 AF38 8B8A D113 FCAC 3C4E 599F

Â»Time flies like an arrow, fruit flies like a Banana.Â«

Eric Abrahamsen

2015-07-07 02:53:02 UTC

Permalink

Post by Nikolaus Rath
Package: gnus
Tags: patch
IMAP servers may sent the data items in a FETCH response in arbitrary
order. However, currently Gnus assumes that the UID item is sent before
the RFC822.SIZE item. If this assumption is wrong, Gnus is unable to
determine the message size.
The attached patch fixes this issue.
Best,
-Nikolaus

Hi Nikolaus,

Would you please resend the patch using git format-patch, and I'll apply
it? With just this patch file, I'd have to manually re-create the
commit, which would lose your metadata as author. Same for your other
two patches, if you don't mind.

Thanks!
Eric

Nikolaus Rath

2015-07-08 04:10:31 UTC

Permalink

Post by Eric Abrahamsen

Hi Nikolaus,
Would you please resend the patch using git format-patch, and I'll apply
it? With just this patch file, I'd have to manually re-create the
commit, which would lose your metadata as author. Same for your other
two patches, if you don't mind.

I just did so. Please let me know if this worked correctly, this is the
first time I've used git.

Best,
-Nikolaus

--
GPG encrypted emails preferred. Key id: 0xD113FCAC3C4E599F
Fingerprint: ED31 791B 2C5C 1613 AF38 8B8A D113 FCAC 3C4E 599F

»Time flies like an arrow, fruit flies like a Banana.«

Eric Abrahamsen

2015-07-11 05:50:06 UTC

Permalink

Post by Nikolaus Rath

Post by Eric Abrahamsen

Hi Nikolaus,
Would you please resend the patch using git format-patch, and I'll apply
it? With just this patch file, I'd have to manually re-create the
commit, which would lose your metadata as author. Same for your other
two patches, if you don't mind.

I just did so. Please let me know if this worked correctly, this is the
first time I've used git.

Okay, I've pushed the size parsing fix as 1f1d39c. The other three
patches are quite a bit more complicated, and I'd like to run them
locally for just a few days just to see if anything explodes. Thanks for
your patience with working on the commits.

Katsumi Yamaoka, thanks for fixing up my commit messages, and sorry to
create extra work! I'll keep trying to get these things formatted
correctly...

Eric