Index: /tags/ipp-unions-20250528/Ohana/src/opihi/pantasks/CheckController.c
===================================================================
--- /tags/ipp-unions-20250528/Ohana/src/opihi/pantasks/CheckController.c	(revision 42915)
+++ /tags/ipp-unions-20250528/Ohana/src/opihi/pantasks/CheckController.c	(revision 42916)
@@ -110,4 +110,5 @@
   /** find "Njobs: **/
   p = memstr (buffer.buffer, "BEGIN BLOCK Njobs:", MIN(buffer.Nbuffer, 256));
+  // XXX if 'p' is NULL, do something sensible here...
   p += strlen("BEGIN BLOCK Njobs:");
 
Index: /tags/ipp-unions-20250528/Ohana/src/opihi/pcontrol/GetJobOutput.c
===================================================================
--- /tags/ipp-unions-20250528/Ohana/src/opihi/pcontrol/GetJobOutput.c	(revision 42915)
+++ /tags/ipp-unions-20250528/Ohana/src/opihi/pcontrol/GetJobOutput.c	(revision 42916)
@@ -2,4 +2,6 @@
 # define PCLIENT_TIMEOUT 100
 # define DEBUG 0
+
+// This function can block for up to 100 * 100,000 nsec = 10 milliseconds (0.01 sec) in each of the two read blocks
 
 // we are trying to read a total of Nbytes from the host.  This function may be called