Serious bug in RefObjectRef::set()

I found a very serious bug in RefObjectRef::set() while fixing a crash in the Terrain Painter (http://www.garagegames.com/mg/forums/result.thread.php?qt=23322):

void RefObjectRef::set(RefBase *object)
{
   decRef();
   mObject = object;
   incRef();
}

If mObject == object and mRefCount == 1 you'll end up with a dangling pointer as the object is freed but the pointer remains set.

The fix is simple:

void RefObjectRef::set(RefBase *object)
   {
      if ( mObject != object ) {
         decRef();
         mObject = object;
         incRef();
      }
   }

With this change you can now change textures in the Terrain Painter without crashing. It was probably effecting other things in the engine also.