infrastructure/mox
1
0
Fork 0
mirror of https://github.com/mjl-/mox.git synced 2025-06-28 08:18:16 +03:00
Code Issues Packages Projects Releases Wiki Activity

imapserver: When trying to replace a message in a non-existent mailbox, do still consume the message if it is a non-synchronized literal

Browse Source 
Not likely to happen in the wild.
This commit is contained in:
Mechiel Lukkien 2025-03-19 22:00:34 +01:00
parent 5294a63c26
commit 9ca50ab207
No known key found for this signature in database
2 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
imapserver/replace.go
View File

@ -77,7 +77,7 @@ func (c *conn) cmdxReplace(isUID bool, tag, cmd string, p *parser) {
return func() { xserverErrorf("finding mailbox: %v", err) } return func() { xserverErrorf("finding mailbox: %v", err) }
} }
if mb == nil { if mb == nil {
xusercodeErrorf("TRYCREATE", "%w", store.ErrUnknownMailbox) return func() { xusercodeErrorf("TRYCREATE", "%w", store.ErrUnknownMailbox) }
} }
// Resolve "*" for UID or message sequence. // Resolve "*" for UID or message sequence.

3
imapserver/replace_test.go
View File

@ -58,6 +58,9 @@ func TestReplace(t *testing.T) {
) )
tc.xcodeArg(imapclient.CodeHighestModSeq(9)) tc.xcodeArg(imapclient.CodeHighestModSeq(9))
// Non-existent mailbox with non-synchronizing literal should consume the literal.
tc.transactf("no", "replace 1 bogusbox {1+}\r\nx")
// Leftover data. // Leftover data.
tc.transactf("bad", "replace 1 inbox () {6+}\r\ntest\r\n ") tc.transactf("bad", "replace 1 inbox () {6+}\r\ntest\r\n ")
} }