comparison audioDB.cpp @ 77:c4389e8f4461

Slightly more bullet-proof server.  In audioDB::error(), We check to see
if we're running as a server (which also includes new audioDB instances
created by the server...); if so, we throw() rather than exit().

 #include "audioDB.h"
 
 #define O2_DEBUG
 
 void audioDB::error(const char* a, const char* b, const char *sysFunc) {
-  cerr << a << ": " << b << endl;
-  if (sysFunc) {
-    perror(sysFunc);
-  }
-  exit(1);
+  if(isServer) {
+    char *err = new char[256]; /* FIXME: overflows */
+    snprintf(err, 255, "%s: %s\n%s", a, b, sysFunc ? strerror(errno) : "");
+    /* FIXME: actually we could usefully do with a properly structured
+       type, so that we can throw separate faultstring and details.
+       -- CSR, 2007-10-01 */
+    throw(err);
+  } else {
+    cerr << a << ": " << b << endl;
+    if (sysFunc) {
+      perror(sysFunc);
+    }
+    exit(1);
+  }
 }
 
 #define O2_AUDIODB_INITIALIZERS \
   dim(0), \
   dbName(0), \

     printf("%s\n", gengetopt_args_info_help[1]);
     printf("%s\n", gengetopt_args_info_help[2]);
     printf("%s\n", gengetopt_args_info_help[0]);
     exit(1);
   }
-  
+
   if(O2_ACTION(COM_SERVER))
     startServer();
 
   else  if(O2_ACTION(COM_CREATE))
     create(dbName);

 }
 
 audioDB::audioDB(const unsigned argc, char* const argv[], adb__queryResult *adbQueryResult): O2_AUDIODB_INITIALIZERS
 {
   processArgs(argc, argv);
+  isServer = 1; // FIXME: Hack
   assert(O2_ACTION(COM_QUERY));
   query(dbName, inFile, adbQueryResult);
 }
 
 audioDB::audioDB(const unsigned argc, char* const argv[], adb__statusResult *adbStatusResult): O2_AUDIODB_INITIALIZERS
 {
   processArgs(argc, argv);
+  isServer = 1; // FIXME: Hack
   assert(O2_ACTION(COM_STATUS));
   status(dbName, adbStatusResult);
 }
 
 audioDB::~audioDB(){

   }
 
   if(args_info.radius_given){
     radius=args_info.radius_arg;
     if(radius<=0 || radius>1000000000){
-      cerr << "Warning: radius out of range" << endl;
-      exit(1);
+      error("radius out of range");
     }
     else 
       if(verbosity>3) {
 	cerr << "Setting radius to " << radius << endl;
       }

 
 // SERVER SIDE
 int adb__status(struct soap* soap, xsd__string dbName, adb__statusResult &adbStatusResult){
   char* const argv[]={"audioDB",COM_STATUS,"-d",dbName};
   const unsigned argc = 4;
-  audioDB(argc, argv, &adbStatusResult);
-  return SOAP_OK;
+  try {
+    audioDB(argc, argv, &adbStatusResult);
+    return SOAP_OK;
+  } catch(char *err) {
+    soap_receiver_fault(soap, err, "");
+    return SOAP_FAULT;
+  }
 }
 
 // Literal translation of command line to web service
 
 int adb__query(struct soap* soap, xsd__string dbName, xsd__string qKey, xsd__string keyList, xsd__string timesFileName, xsd__int qType, xsd__int qPos, xsd__int pointNN, xsd__int trackNN, xsd__int seqLen, adb__queryResult &adbQueryResult){