Author: rdonkin
Date: Sat Nov 17 04:30:01 2007
New Revision: 595941
URL: http://svn.apache.org/viewvc?rev=595941&view=rev
Log:
Fix buggy locks
Modified:
james/server/trunk/torque-mailboxmanager-function/src/main/java/org/apache/james/mailboxmanager/torque/TorqueMailbox.java
Modified:
james/server/trunk/torque-mailboxmanager-function/src/main/java/org/apache/james/mailboxmanager/torque/TorqueMailbox.java
URL:
http://svn.apache.org/viewvc/james/server/trunk/torque-mailboxmanager-function/src/main/java/org/apache/james/mailboxmanager/torque/TorqueMailbox.java?rev=595941&r1=595940&r2=595941&view=diff
==============================================================================
---
james/server/trunk/torque-mailboxmanager-function/src/main/java/org/apache/james/mailboxmanager/torque/TorqueMailbox.java
(original)
+++
james/server/trunk/torque-mailboxmanager-function/src/main/java/org/apache/james/mailboxmanager/torque/TorqueMailbox.java
Sat Nov 17 04:30:01 2007
@@ -130,7 +130,7 @@
public MessageResult appendMessage(MimeMessage message, Date internalDate,
int result) throws MailboxManagerException {
try {
- lock.readLock().acquire();
+ lock.writeLock().acquire();
try {
checkAccess();
final MailboxRow myMailboxRow;
@@ -212,7 +212,7 @@
throw new MailboxManagerException("Mailbox has been
deleted");
}
} finally {
- lock.readLock().release();
+ lock.writeLock().release();
}
} catch (InterruptedException e) {
throw new MailboxManagerException(e);
@@ -734,13 +734,13 @@
public synchronized long getUidValidity() throws MailboxManagerException {
try {
- lock.writeLock().acquire();
+ lock.readLock().acquire();
try {
checkAccess();
final long result = getMailboxRow().getUidValidity();
return result;
} finally {
- lock.writeLock().release();
+ lock.readLock().release();
}
} catch (InterruptedException e) {
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
