diff --git a/Lib/mailbox.py b/Lib/mailbox.py index 4da556935ece..2b7b14e2f1aa 100755 --- a/Lib/mailbox.py +++ b/Lib/mailbox.py @@ -892,17 +892,9 @@ def remove(self, key): raise KeyError('No message with key: %s' % key) else: raise - try: - if self._locked: - _lock_file(f) - try: - f.close() - os.remove(os.path.join(self._path, str(key))) - finally: - if self._locked: - _unlock_file(f) - finally: + else: f.close() + os.remove(path) def __setitem__(self, key, message): """Replace the keyed message; raise KeyError if it doesn't exist.""" diff --git a/Lib/test/test_mailbox.py b/Lib/test/test_mailbox.py index c88af0613722..7575ca7e65e6 100644 --- a/Lib/test/test_mailbox.py +++ b/Lib/test/test_mailbox.py @@ -979,6 +979,13 @@ def test_issue2625(self): key0 = self._box.add(msg0) refmsg0 = self._box.get_message(key0) + def test_issue7627(self): + msg0 = mailbox.MHMessage(self._template % 0) + key0 = self._box.add(msg0) + self._box.lock() + self._box.remove(key0) + self._box.unlock() + def test_pack(self): # Pack the contents of the mailbox msg0 = mailbox.MHMessage(self._template % 0)