Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Deleting a non-existing folder throws a fatal error #4997

Open
alecpl opened this issue Aug 12, 2024 · 1 comment
Open

Deleting a non-existing folder throws a fatal error #4997

alecpl opened this issue Aug 12, 2024 · 1 comment

Comments

@alecpl
Copy link
Contributor

alecpl commented Aug 12, 2024

Using Cyrus IMAP 3.6.6.

C: A0014 DELETE Folder/Test
S: * BYE Fatal error: Mailbox has been (re)moved

It should not cause BYE response.

cyrus-imapd/imap[3690]: command: A0014 Delete
cyrus-imapd/imap[3690]: Rename: kolab.org!user.jack.Folder.Test -> kolab.org!DELETED.user.jack.Folder.Test.66B692A7
cyrus-imapd/imap[3690]: Mailbox kolab.org!user.jack.Folder.Test has been (re)moved out from under client
cyrus-imapd/imap[3690]: Fatal error: Mailbox has been (re)moved
cyrus-imapd/master[1]: process type:SERVICE name:imap path:/usr/libexec/imapd age:99.868s pid:3690 exited, status 74
@elliefm
Copy link
Contributor

elliefm commented Aug 28, 2024

I've created a Cassandane test (#5010) to try to reproduce this, but haven't yet been able to. I wonder if I haven't properly reproduced the circumstances? Or perhaps the issue comes from a Kolab patch.

The test first tries deleting some mailboxes that have never existed, then creates them, then deletes them for real, then tries deleting them again. The responses are all sane:

C: 4 delete Foo
S: 4 NO Mailbox does not exist
C: 5 delete Foo/bar
S: 5 NO Mailbox does not exist
C: 6 delete Foo/bar/baz
S: 6 NO Mailbox does not exist
C: 7 create Foo
S: 7 OK [MAILBOXID (2b7fd5f5-42c1-43d5-b603-93a80c8e08d3)] Completed
C: 8 create Foo/bar
S: 8 OK [MAILBOXID (2713b0a9-3c47-494f-9500-0ba6175d66a1)] Completed
C: 9 create Foo/bar/baz
S: 9 OK [MAILBOXID (27b72a06-d965-4747-8b5f-4bb3b18f5f79)] Completed
C: 10 delete Foo/bar/baz
S: 10 OK Completed
C: 11 delete Foo/bar
S: 11 OK Completed
C: 12 delete Foo
S: 12 OK Completed
C: 13 delete Foo
S: 13 NO Mailbox does not exist
C: 14 delete Foo/bar
S: 14 NO Mailbox does not exist
C: 15 delete Foo/bar/baz
S: 15 NO Mailbox does not exist

If you're able to run Cassandane tests, can you able to apply this patch and see if the test passes or fails for you?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants