Changeset 20059 for trunk/Ohana/src/opihi/pcontrol/CheckBusyJob.c

Timestamp:

Oct 10, 2008, 3:28:11 PM (18 years ago)

Author:

eugene

Message:

adding some ASSERTS to catch a thief

File:

• trunk/Ohana/src/opihi/pcontrol/CheckBusyJob.c (modified) (4 diffs)

trunk/Ohana/src/opihi/pcontrol/CheckBusyJob.c

@@ -57 +57 @@
   buffer = &host[0].comms_buffer;
   job = (Job *) host[0].job;
+  ASSERT (host == (Host *) job[0].host, "invalid host");
 
   /** host is up, need to parse message **/
@@ -87 +88 @@
     return (TRUE);
   }
-
+  ASSERT (outstate != PCONTROL_JOB_DONE, "impossible outstate for job");
 
   /* parse the exit status and sizes of output buffers */
@@ -97 +98 @@
   sscanf (p, "%*s %d", &job[0].stderr.size);
 
+  ASSERT (job[0].stdout.size > 0, "job has no output?");
+  ASSERT (job[0].stderr.size > 0, "job has no output?");
+
   // XXX runaway job if output too large?
   if (job[0].stdout.size > 0x1000000) abort();
@@ -105 +109 @@
   // don't unlink job and host yet
   PutHost (host, PCONTROL_HOST_BUSY, STACK_BOTTOM);
+  ASSERT (job[0].host, "host not set for job");
   PutJobSetState (job, PCONTROL_JOB_DONE, STACK_BOTTOM, outstate);
   gettimeofday (&job[0].stop, NULL);