fix copy ctor

Using operator= in a copy ctor is a rather bad idea when the operator= accesses
uninitialized data members.

1 files changed, 7 insertions, 5 deletions

diff --git a/connectivity/source/drivers/ado/Awrapado.cxx b/connectivity/source/drivers/ado/Awrapado.cxx
index e7608747edce..08535858cf55 100644
--- a/connectivity/source/drivers/ado/Awrapado.cxx
+++ b/connectivity/source/drivers/ado/Awrapado.cxx
@@ -1561,6 +1561,13 @@ WpBase::WpBase(IDispatch* pInt)
     }
 }
 
+WpBase::WpBase(const WpBase& aWrapper)
+    :pIUnknown(aWrapper.pIUnknown)
+{
+    if (pIUnknown)
+        pIUnknown->AddRef();
+}
+
 //inline
 WpBase& WpBase::operator=(const WpBase& rhs)
 {
@@ -1588,11 +1595,6 @@ WpBase& WpBase::operator=(IDispatch* rhs)
     return *this;
 }
 
-WpBase::WpBase(const WpBase& aWrapper)
-{
-    operator=(aWrapper);
-}
-
 WpBase::~WpBase()
 {
     if (pIUnknown)