Errata

CookieStore.java:
add:
if( null == setCookies)
return;
before:
for (String next : setCookies) {
try {
Cookie cookie = Cookie.bake(next, uri);
...
or there will be exception when server does not set cookies.

Note from the Author or Editor:
On p. 273 immediately after the line
List<String> setCookies = repsonseHeaders.get("Set-Cookie");

add this line:

if (setCookies == null) return;

The paragraph at the top of the page should read:

one additional field, an InstanceCallbackDigestUserInterface object called callback. At the end of
the run() method, the digest is passed to callback's receiveDigest() method. The
InstanceCallbackDigestUserInterface object itself is set in the constructor.

The CallbackDigestUserInterface class shown in Example 5-8
to
The InstanceCallbackDigestUserInterface class shown in Example 5-8

NetworkInterface.getByName(local) to
NetworkInterface.getByInetAddress(local)

First, change new BufferedReader to new SafeBufferedReader.
Then in the ,code>main() method, change e.printStackTrace() to ex.printStackTrace().
Finally, several lines from the processLogFile() method were omitted.
The complete, corrected program is:

import java.io.*;
import java.util.*;
import com.macfaq.io.SafeBufferedReader;

public class PooledWeblog {

private BufferedReader in;
private BufferedWriter out;
private int numberOfThreads;
private List entries = Collections.synchronizedList(new LinkedList());
private boolean finished = false;
private int test = 0;

public PooledWeblog(InputStream in, OutputStream out, int
numberOfThreads) {
this.in = new SafeBufferedReader(new InputStreamReader(in));
this.out = new BufferedWriter(new OutputStreamWriter(out));
this.numberOfThreads = numberOfThreads;
}

public boolean isFinished() {
return this.finished;
}

public int getNumberOfThreads() {
return numberOfThreads;
}

public void processLogFile() {

for (int i = 0; i < numberOfThreads; i++) {
Thread t = new LookupThread(entries, this);
t.start();
}

try {
String entry = in.readLine();
while (entry != null) {

if (entries.size() > numberOfThreads) {
try {
Thread.sleep((long) (1000.0/numberOfThreads));
}
catch (InterruptedException e) {}
continue;
}

synchronized (entries) {
entries.add(0, entry);
entries.notifyAll();
}

entry = in.readLine();
Thread.yield();

} // end while

}
catch (IOException e) {
System.out.println("Exception: " + e);
}

this.finished = true;

// finish any threads that are still waiting
synchronized (entries) {
entries.notifyAll();
}

}

public void log(String entry) throws IOException {
out.write(entry + System.getProperty("line.separator", "
"));
out.flush();
}

public static void main(String[] args) {

try {
PooledWeblog tw = new PooledWeblog(new FileInputStream(args[0]),
System.out, 100);
tw.processLogFile();
}
catch (FileNotFoundException e) {
System.err.println("Usage: java PooledWeblog logfile_name");
}
catch (ArrayIndexOutOfBoundsException e) {
System.err.println("Usage: java PooledWeblog logfile_name");
}
catch (Exception ex) {
System.err.println(ex);
ex.printStackTrace();
}

} // end main

}

"Finally, the name-value pairs are simply the NAME attributes of the INPUT elements, except for any INPUT elements whose
TYPE attributes has the value submit."
to
"Finally, the names in the name-value pairs are simply the values of the NAME attributes of the
INPUT elements. The values of the pairs are whatever the user types into the form."

In Example 8-11, the matches() method should also check the domain. That is, it should read:

public boolean matches(URI u) {

if (isExpired()) return false;

String path = u.getPath();
if (path == null) path = "/";

String domain = u.getHost();

if (path.startsWith(this.path)
&& domain.equals(thisdomain)) {
return true;
}

return false;
}

Change
if (s.getTcpSoLinger() == -1) s.setSoLinger(true, 240);
to
if (s.getSoLinger() == -1) s.setSoLinger(true, 240);

Code example reads:

} catch(IOException ex) {
e.PrintStackTrace();
}

There is no "e" in that Exception...

Note from the Author or Editor:
Change e.printStackTrace to ex.printStackTrace on p. 343

"if (args.length >= 2)" should be "if (args.length > 2)".

The block of the second 'catch' uses the variable 'e' instead of 'ex'.
So the code should read:
catch (IOException ex) {
System.out.println("Server could not start because of an "
+ ex.getClass());
System.out.println(ex);
}

"java.policy" should be "java.security" in both paragraphs

I noticed a small error in O'Reilly Java Network Programming 3rd Edition.
The error is at the beginning of page 368 of the hardcopy.

I think the method public abstract Socket createSocket(InetAddress host, int port,
InetAddress interface, int localPort) shouldn't throw UnknownHostException.

Although the next method public abstract Socket createSocket(Socket proxy, String
host, int port, boolean autoClose) should throw that UnknownHostException.

So I think the two methods should be printed like this:-

public abstract Socket createSocket(InetAddress host, int port, InetAddress
interface, int localPort) throws IOException

public abstract Socket createSocket(Socket proxy, String host, int port, boolean
autoClose) throws IOException, UnknownHostException


Thanks again for that nice book.

Tareq Jahhaf.

Note from the Author or Editor:
On p. 368 remove ", UnknownHostExeption" from the third line and place it at the end of the fifth line (right after IOException)

There are two fields in the diagram both shown as "destination port (0-65535)". There should
be only one such field, the 2nd of the duplicates should instead be indicated as being a checksum.

The last two constructors for the DatagramPacket class have typos. In particular, these last two should take a SocketAddress destination as the last parameter, not have have int port as the last parameter. So they should look like:

DatagramPacket(byte[] buf, int length,
SocketAddress address)

DatagramPacket(byte[] buf, int offset, int length,
SocketAddress address)

FYI, I cut the definitions above directly from the Javadoc at http://download.oracle.com/javase/1,5.0/docs/api/index.html?java/net/InetSocketAddress.html

Regards,
Mike

Note from the Author or Editor:
This erratum is correct. On p. 428 change ", int port) // Java 1.4" to simply ") // Java 1.4" both places it occurs (in the 6th and 8th lines of the code fragment at the top of the page.

they're using the local host's default encoding. However, is these two
programs run of different hosts (for instance, one is a Japanese system
and one is a French system) that may well not be the same for each. The
encoding needs to be explicitly specified in both programs. On p. 439 in
Example 13-3 change

byte[] data = theLine.getBytes()
to
byte[] data = theLine.getBytes("UTF-8")

On p. 441 in Example 13-4 change

String s = new String(packet.getData(), 0, packet.getLength())
to
String s = new String(packet.getData(), 0, packet.getLength(), "UTF-8")

This will allow the two hosts to exchange data regardless of differing encodings.

It's in the hardcopy on page 444 in paragraph 3.
The title of the paragraph is:-
public InetAddress getRemoteSocketAddress( )
I think the right should be:-
public SocketAddress getRemoteSocketAddress( )
The method getRemoteSocketAddress() is in class SocketAddress not in InetAddress.

Note from the Author or Editor:
Change public InetAddress getRemoteSocketAddress( )
to public SocketAddress getRemoteSocketAddress( )

1 to 9. That is,

for (int i = 1; i < 10; i++) {

should be:

for (int i = 1; i <= 10; i++) {

public abstract class CacheRequest

should be

public abstract class CacheResponse