The destructor always assumed that it would be called on objects created from

ONE constructor routine.  This was bogus - there is also a system provided
constructor which does not allocate a pattern buffer.  Make less assumptions.

1 files changed, 7 insertions, 4 deletions

diff --git a/gnu/lib/libg++/libg++/Regex.cc b/gnu/lib/libg++/libg++/Regex.cc
index 37e564b..65f03a0 100644
--- a/gnu/lib/libg++/libg++/Regex.cc
+++ b/gnu/lib/libg++/libg++/Regex.cc
@@ -39,10 +39,13 @@ extern "C" {
 
 Regex::~Regex()
 {
-  if (buf->buffer) free(buf->buffer);
-  if (buf->fastmap) free(buf->fastmap);
-  delete(buf);
-  delete(reg);
+  if (buf) {
+  	if (buf->buffer) free(buf->buffer);
+  	if (buf->fastmap) free(buf->fastmap);
+  	delete(buf);
+  }
+  if (reg)
+  	delete(reg);
 }
 
 Regex::Regex(const char* t, int fast, int bufsize,