[BOOKKEEPER-924] addEntry() is susceptible to spurious wakeups

• Type: Bug
• Status: Resolved
• Priority: Critical
• Resolution: Fixed
• Affects Version/s: 4.3.2
• Fix Version/s: 4.5.0
• Assignee: Enrico Olivelli
• Reporter: JV Jujjuri
• Due: 27/May/16
• Created: 22/Apr/16 16:40
• Updated: 16/Oct/16 15:09
• Resolved: 13/Oct/16 07:27

Description

LedgerHandle sync interface heavily depends on SyncCounter to convert async interfaces
into sync interfaces.

Usaylly

SyncCounter.inc()
asyncCall()
SyncCOunter.block(0)

The block code is.

synchronized void block(int limit) throws InterruptedException {
while (i > limit) {
int prev = i;
wait();
if (i == prev)

{ break; }

}
}

Since 'i' is going to be same as 'prev' on spurious wakeup, and wait() can return on spurious wakeups.

Issue Links

• links to: GitHub Pull Request #38
• links to: GitHub Pull Request #60