Temporarily reverted patch for bug #41868 as it was causing problems in PB.

1 files changed, 10 insertions, 10 deletions

diff --git a/sql/sql_string.cc b/sql/sql_string.cc
index b6ce4d8dc8d..75e47dd0c8e 100644
--- a/sql/sql_string.cc
+++ b/sql/sql_string.cc
@@ -72,26 +72,26 @@ bool String::realloc(uint32 alloc_length)
     if (alloced)
     {
       if ((new_ptr= (char*) my_realloc(Ptr,len,MYF(MY_WME))))
-        new_ptr[alloc_length]= 0;
+      {
+	Ptr=new_ptr;
+	Alloced_length=len;
+      }
       else
-        return TRUE;				// Signal error
+	return TRUE;				// Signal error
     }
     else if ((new_ptr= (char*) my_malloc(len,MYF(MY_WME))))
     {
-      if (str_length > len - 1)
-        str_length= 0;
       if (str_length)				// Avoid bugs in memcpy on AIX
-        memcpy(new_ptr, Ptr, str_length);
-      new_ptr[str_length]= 0;
+	memcpy(new_ptr,Ptr,str_length);
+      new_ptr[str_length]=0;
+      Ptr=new_ptr;
+      Alloced_length=len;
       alloced=1;
     }
     else
       return TRUE;			// Signal error
-    Ptr= new_ptr;
-    Alloced_length= len;
   }
-  else
-    Ptr[alloc_length]= 0;
+  Ptr[alloc_length]=0;			// This make other funcs shorter
   return FALSE;
 }