Improve liboop error checking. Increase PROTECTED_FDS, since liboop

uses internal file descriptors.

ChangeLog

 2003-07-14  Per Cederqvist  <ceder@ceder.dyndns.org>
 
+	Improve liboop error checking.
+	* src/libraries/liboop/signal.c (add_flag): New static function,
+	that checks the return values from fcntl properly.
+	(oop_signal_new): Use add_flag() to simplify code. Fail (return
+	NULL) if any of the fcntl calls on the new pipe fails.
+
+	Increase PROTECTED_FDS, since liboop uses internal file
+	descriptors.
+	* src/include/kom-config.h (PROTECTED_FDS): Increased from 12 to
+	20.  Document that liboop uses a couple of file descriptors, and
+	that this setting is also present in unix.exp.
+	* src/server/testsuite/config/unix.exp (PROTECTED_FDS): New
+	constant.
+	* src/server/testsuite/lyskomd.0/07.exp: Use PROTECTED_FDS.
+	* src/server/testsuite/lyskomd.0/connections-cov.exp: Ditto.
+	* src/server/testsuite/lyskomd.0/prot-a-send-async-cov.exp: Ditto.
+	* src/server/testsuite/lyskomd.0/send-async-cov.exp: Ditto.
+
 	Flush the output less often.  This fixes the non-controversial
 	parts of bug 107.
 	* src/server/prot-a.c (prot_a_reply): Don't call isc_flush()