Changeset 21455 for main/trunk/greenstone2/runtime-src/src/colservr

Timestamp:

2010-01-14T19:34:40+13:00 (14 years ago)

Author:

ak19

Message:

Major bug fix after Dr Bainbridge debugged the server code which was causing the server to crash upon Enter Library: the line where it failed was simply where a local-scope text_t object was being naturally destructed. The real error was a memory corruption which the use of the new heapchk function helped to locate: collectset's inherited method configure() ought to have been specified as virtual, since it was declared virtual in the superclass configurable. A couple of destructors (in collectset and oaiservr's oaiconfig) have been made virtual as well since the superclass configurable has virtual methods which requires all related destructors to be virtual.

File:

• main/trunk/greenstone2/runtime-src/src/colservr/collectset.h (modified) (3 diffs)

main/trunk/greenstone2/runtime-src/src/colservr/collectset.h

@@ -35 +35 @@
 #endif
 
-  int                   noofservers;
   text_t                httpdomain;
   text_t                httpprefix;
@@ -48 +47 @@
   collectset();
 
-  ~collectset();
+  virtual ~collectset();
 
   // initialise the collections
@@ -54 +53 @@
 
   // configure all the servers as supplied
-  void configure (const text_t &key, const text_tarray &cfgline);
+  virtual void configure (const text_t &key, const text_tarray &cfgline);
 
   // Add/remove collections