check for self-assignment

Change-Id: I74eb934d19c0b511870e5a675917ca4baaf768cc

1 files changed, 11 insertions, 8 deletions

diff --git a/editeng/source/items/frmitems.cxx b/editeng/source/items/frmitems.cxx
index 6bfcbaa1bfd6..ec383ebce144 100644
--- a/editeng/source/items/frmitems.cxx
+++ b/editeng/source/items/frmitems.cxx
@@ -3674,14 +3674,17 @@ bool SvxBrushItem::GetPresentation
 
 SvxBrushItem& SvxBrushItem::operator=(const SvxBrushItem& rItem)
 {
-    aColor = rItem.aColor;
-    nShadingValue = rItem.nShadingValue;
-    xGraphicObject.reset(rItem.xGraphicObject ? new GraphicObject(*rItem.xGraphicObject) : nullptr);
-    nGraphicTransparency = rItem.nGraphicTransparency;
-    maStrLink = rItem.maStrLink;
-    maStrFilter = rItem.maStrFilter;
-    eGraphicPos = rItem.eGraphicPos;
-    bLoadAgain = rItem.bLoadAgain;
+    if (&rItem != this)
+    {
+        aColor = rItem.aColor;
+        nShadingValue = rItem.nShadingValue;
+        xGraphicObject.reset(rItem.xGraphicObject ? new GraphicObject(*rItem.xGraphicObject) : nullptr);
+        nGraphicTransparency = rItem.nGraphicTransparency;
+        maStrLink = rItem.maStrLink;
+        maStrFilter = rItem.maStrFilter;
+        eGraphicPos = rItem.eGraphicPos;
+        bLoadAgain = rItem.bLoadAgain;
+    }
     return *this;
 }