In the following code fragment occurring twice in zoo_service_loader.c, GetProcAddress or dlsym will return NULL if a user has mistyped the service name (the WPS Identifier parameter):

#ifdef WIN32
	typedef int (CALLBACK* execute_t)(char***,char***,char***);
	execute_t execute=(execute_t)GetProcAddress(so,fname);
#else
	typedef int (*execute_t)(char***,char***,char***);
	execute_t execute=(execute_t)dlsym(so,fname);
#endif

This will cause a crash when attempting to invoke the function at the address given by execute:

*eres=execute(&m,&request_input_real_format,&request_output_real_format);

There should be a check to avoid this happening (with an error message).

On CENTOS 6 cd ../zoo-services/ogr/base-vect-ops/ make clean rm -f cgi-env/ogr_service.zo

make g++ -I../../../../thirds/cgic206/ -I../../../zoo-kernel/ -I/usr/include/libxml2 -I/usr/include/gdal geos-config --cflags -DLINUX_FREE_ISSUE -shared -fpic -o cgi-env/ogr_service.zo ./service.c ../../../zoo-kernel/service_internal.o -L/usr/lib64 -lgdal -lxml2 -lz -lm -lcrypto geos-config --libs ./service.c:46: erreur: declaration of C function ‘int errorException(maps*, const char*, const char*)’ conflicts with ../../../zoo-kernel/service_internal.h:139: erreur: previous declaration ‘int errorException(maps*, const char*, const char*, const char*)’ here ./service.c: In function ‘void* createGeometryFromGML(maps*, char*)’: ./service.c:55: attention : deprecated conversion from string constant to ‘char*’ ./service.c: In function ‘int Boundary(maps*&, maps*&, maps*&)’: ./service.c:343: attention : deprecated conversion from string constant to ‘char*’ ./service.c: In function ‘int ConvexHull?(maps*&, maps*&, maps*&)’: ./service.c:350: attention : deprecated conversion from string constant to ‘char*’ ./service.c: In function ‘int Centroid(maps*&, maps*&, maps*&)’: ./service.c:369: attention : deprecated conversion from string constant to ‘char*’ make: * [cgi-env/ogr_service.zo] Erreur 1

The following lines in the function printFullDescription (service_internal.c) causes a segmentation fault in some situations, namely if the pointer e->supported is NULL (tested on Windows 7):

if(_tmp0==NULL){
   xmlAddChild(nc6,nc8);
   _tmp0=e->supported;
   if(getMap(_tmp0->content,"range")!=NULL ||
      getMap(_tmp0->content,"rangeMin")!=NULL ||
      getMap(_tmp0->content,"rangeMax")!=NULL ||
      getMap(_tmp0->content,"rangeClosure")!=NULL ) { 
      tmp1=_tmp0->content;
      goto doRange;
   }
}