Andrew book persistDvd method ERROR?

[Karol Wiszowaty]

Hi all!

There is some strange code in Andrew's book project soure code. I think the method DvdFileAccess.persistDvd() has error.
When you are adding new record the offset is calculating:
<blockquote>code:

---

<pre name="code" class="core">
offset = database.length();
</pre>

---

</blockquote>
But there is no lock on database on that time. Another thread can easily enter the method and get the same offset beacuse writing to database is made some lines farther:
<blockquote>code:

---

<pre name="code" class="core">
synchronized (database) {
database.seek(offset);
database.write(out.toString().getBytes());
}
</pre>

---

</blockquote>
So another thread can rewrite new record.

Am I right?
Thanks for help.
[ July 16, 2008: Message edited by: Karol Wiszowaty ]

[Shengshuo Wu]

Hi Karol,

Hmm... I see your point.
[ July 16, 2008: Message edited by: Shengshuo Wu ]

[Jeffry Kristianto Yanuar]

Hi, check out for the book's errata in

http://apress.com/book/downloadfile/2611

Jeffry Kristianto Yanuar
SCJP 5.0
SCJA
SCJD (Working on UrlyBird 1.3.2 : almost done)

[Karol Wiszowaty]

Thanks for help and link to Errata.