Changeset 57 for trunk/zoo-kernel/service_conf.y

Timestamp:

Jan 5, 2011, 3:20:01 PM (13 years ago)

Author:

djay

Message:

Adding contains and getIoTypeFromElement functions in service.h. Remove memory leaks from the Service Configuration Parser, add the support for multiple supported formats as before. Small modification of the ZOO-Kernel Java Support, adding the capability to access the modified main_conf HashMap? from the Kernel to let lenv message map pass when an error occurs and it is handled in the Service code. Adding same capability for the ZOO-Kernel Python Support. Use strcasecmp in service.h rather than strlen+strncasecmp. Ensure that only OWS compliant informations are available for Contact.Phone and Contact.Adress. Remove memory leak in createExceptionReportNode. Correction of the addDefaultValues function to add the default format using the informations from DataInputs? if present, this should correct the behavior of the ZOO-Kernel when choosing the extension value which should now point to the corresponding zcfg value if present. Don't set the NULL value for inputs not provided in the DataInputs?, still set NULL as default value for outputs. Avoid segfault in freeElements when some zcfg values was not set correctly. Link against the client libjvm.so file rather than the server one.

File:

• trunk/zoo-kernel/service_conf.y (modified) (15 diffs)

trunk/zoo-kernel/service_conf.y

@@ -133 +133 @@
  : STag contentetoile ETag      
 {
-  free($3);
 }
 
@@ -210 +209 @@
           my_service->inputs->next=NULL;
         }
-        else
+        else if(current_element!=NULL && current_element->name!=NULL){
           addToElements(&my_service->inputs,current_element);
+        }
 #ifdef DEBUG_SERVICE_CONF
         fprintf(stderr,"CURRENT_ELEMENT\n");
@@ -267 +267 @@
               addMapToMap(&current_element->metadata,current_content);
               current_element->next=NULL;
-              current_element->format=$2;
+              current_element->format=strdup($2);
               current_element->defaults=NULL;
               current_element->supported=NULL;
@@ -280 +280 @@
             current_element->metadata=NULL;
             current_element->next=NULL;
-            current_element->format=$2;
+            current_element->format=strdup($2);
             current_element->defaults=NULL;
             current_element->supported=NULL;
@@ -297 +297 @@
               if(wait_defaults==true){
                 defaultsc++;
-                freeMap(&current_content);
-                current_content=NULL;
+                /*freeMap(&current_content);
+                  current_content=NULL;*/
               }
               current_data=5;
@@ -305 +305 @@
   printf("* Identifiant : %s\n",$2);
 #endif
-  /* et on renvoie l'identifiant de balise afin de pouvoir le comparer */
-  /* avec la balise jumelle fermante ! */
-  $$ = $2 ;
 }
  ;
@@ -338 +335 @@
   printf ("attribute : %s\n",$1) ;
 #endif
-  free($1);
 }
  ;
@@ -391 +387 @@
     current_content=NULL;
     current_element->supported=NULL;
+    current_element->next=NULL;
   }
   if(strcmp($3,"Supported")==0){
     current_data=previous_data;
     if(current_element->supported==NULL){
-      //addMapToIoType(&current_element->supported,current_content);
-      current_element->supported=(iotype*)malloc(IOTYPE_SIZE);
-      current_element->supported->content=NULL;
-      addMapToMap(&current_element->supported->content,current_content);
-      freeMap(&current_content);
-      free(current_content);
-      current_element->supported->next=NULL;
-      current_content=NULL;
+      if(current_content!=NULL){
+        current_element->supported=(iotype*)malloc(IOTYPE_SIZE);
+        current_element->supported->content=NULL;
+        addMapToMap(&current_element->supported->content,current_content);
+        freeMap(&current_content);
+        free(current_content);
+        current_element->supported->next=NULL;
+        current_content=NULL;
+      }else{
+        current_element->supported=NULL;
+        current_element->next=NULL;
+      }
     }
     else{
-#ifdef DEBUG
-      // Currently we support only one supported format
+#ifdef DEBUG_SERVICE_CONF
       fprintf(stderr,"SECOND SUPPORTED FORMAT !!!!\n");
 #endif
-      //addMapToIoType(&current_element->supported,current_content);
-      /*iotype* iotmp=*(&current_element->supported);
-      while(iotmp!=NULL){
-        dumpMap(iotmp->content);
-        iotmp=iotmp->next;
-      }
-      iotmp=(iotype*)malloc(IOTYPE_SIZE);
-      iotmp->content=NULL;
-      addMapToMap(&iotmp->content,current_content);
-      iotmp->next=NULL;
-      dumpElements(current_element);
-      fprintf(stderr,"SECOND SUPPORTED FORMAT MAP START !!!!\n");
-      dumpMap(current_content);
-      fprintf(stderr,"SECOND SUPPORTED FORMAT MAP END !!!!\n");*/
+      addMapToIoType(&current_element->supported,current_content);
       freeMap(&current_content);
       free(current_content);
       current_content=NULL;
-      /*freeMap(&iotmp->content);
-      free(&iotmp->content);
-      free(iotype);*/
-#ifdef DEBUG
-      // Currently we support only one supported format
+#ifdef DEBUG_SERVICE_CONF
+      dumpElements(current_element);
       fprintf(stderr,"SECOND SUPPORTED FORMAT !!!!\n");
 #endif
@@ -436 +420 @@
     current_content=NULL;
   }
-  /* on renvoie l'identifiant de la balise pour pouvoir comparer les 2 */
-  /* /!\ une balise fermante n'a pas d'attributs (c.f. : W3C) */
-  $$ = $3;
 }
  ;
@@ -610 +591 @@
       if(current_data==2){ 
         if(wait_inputs==true){
-          fprintf(stderr,"dup INPUTS\n");
-          if(current_element->name!=NULL){
+          if(current_element!=NULL && current_element->name!=NULL){
             if(my_service->inputs==NULL){
               my_service->inputs=dupElements(current_element);
@@ -661 +641 @@
           }
           wait_inputs=false;
-          dumpMap(current_content);
           current_content=NULL;
         }
@@ -755 +734 @@
   int resultatYYParse = srparse() ;
   
-  if(wait_outputs==true && current_element->name!=NULL){
+  if(wait_outputs==true && current_element!=NULL && current_element->name!=NULL){
     if(my_service->outputs==NULL){      
 #ifdef DEBUG_SERVICE_CONF
@@ -762 +741 @@
       my_service->outputs=dupElements(current_element);
       my_service->outputs->next=NULL;
-      freeElements(&current_element);
-      free(current_element);
-      current_element=NULL;
     }
     else{
@@ -778 +754 @@
     free(current_element);
     current_element=NULL;
+#ifdef DEBUG_SERVICE_CONF
+    fprintf(stderr,"(DATAOUTPUTS - 631) FREE current_element\n");
+#endif
   }
   if(current_element!=NULL){
     freeElements(&current_element);
-    fprintf(stderr,"LINE 709");
     free(current_element);
     current_element=NULL;
@@ -787 +765 @@
   if(current_content!=NULL){
     freeMap(&current_content);
-    fprintf(stderr,"LINE 715");
     free(current_content);
     current_content=NULL;