xmlsecurity: clean up unused mxSecurityController

Change-Id: Ife64ab3683479baf152357a6167718f13c9b6089
Reviewed-on: https://gerrit.libreoffice.org/30964
Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>
Tested-by: Jenkins <ci@libreoffice.org>

3 files changed, 3 insertions, 10 deletions

diff --git a/xmlsecurity/inc/xmlsignaturehelper.hxx b/xmlsecurity/inc/xmlsignaturehelper.hxx
index d8f97e5a9b16..79311a7fe14c 100644
--- a/xmlsecurity/inc/xmlsignaturehelper.hxx
+++ b/xmlsecurity/inc/xmlsignaturehelper.hxx
@@ -24,7 +24,9 @@
 
 #include <tools/link.hxx>
 #include <rtl/ustring.hxx>
+#include <rtl/ref.hxx>
 #include <sigstruct.hxx>
+#include <xsecctl.hxx>
 #include <xmlsecuritydllapi.h>
 
 #include <com/sun/star/uno/XComponentContext.hpp>
@@ -35,7 +37,6 @@
 #include <com/sun/star/xml/crypto/sax/XSignatureCreationResultListener.hpp>
 #include <com/sun/star/xml/crypto/sax/XSignatureVerifyResultListener.hpp>
 
-class XSecController;
 class Date;
 namespace tools { class Time; }
 
@@ -91,20 +92,13 @@ class XMLSECURITY_DLLPUBLIC XMLSignatureHelper
 {
 private:
     css::uno::Reference< css::uno::XComponentContext > mxCtx;
-    // FIXME: This field appears to be unused (it is only assigned to in the XMLSignatureHelper
-    // constructor), but it can't be removed as it is that very assignment which causes the object
-    // it to be acquired, and otherwise its reference counting will be borked and we get a
-    // crash. This is stupid of course. Probably we should just kill the separate XSecController
-    // class and move its contents inside this class. Nothing else uses XSecController anyway, as
-    // far as I see.
-    css::uno::Reference< css::xml::crypto::sax::XSecurityController > mxSecurityController;
     css::uno::Reference< css::xml::crypto::XUriBinding > mxUriBinding;
 
     std::vector<XMLSignatureCreationResult>
                                 maCreationResults;
     std::vector<XMLSignatureVerifyResult>
                                 maVerifyResults;
-    XSecController*             mpXSecController;
+    rtl::Reference<XSecController> mpXSecController;
     bool                        mbError;
     bool mbODFPre1_2;
     Link<LinkParamNone*,bool>   maStartVerifySignatureHdl;
diff --git a/xmlsecurity/source/helper/xsecctl.hxx b/xmlsecurity/inc/xsecctl.hxx
index 9cbb9e0c6159..9cbb9e0c6159 100644
--- a/xmlsecurity/source/helper/xsecctl.hxx
+++ b/xmlsecurity/inc/xsecctl.hxx
diff --git a/xmlsecurity/source/helper/xmlsignaturehelper.cxx b/xmlsecurity/source/helper/xmlsignaturehelper.cxx
index a4243d9e63a0..36a6117c29a0 100644
--- a/xmlsecurity/source/helper/xmlsignaturehelper.cxx
+++ b/xmlsecurity/source/helper/xmlsignaturehelper.cxx
@@ -61,7 +61,6 @@ XMLSignatureHelper::XMLSignatureHelper( const uno::Reference< uno::XComponentCon
     : mxCtx(rxCtx), mbODFPre1_2(false)
 {
     mpXSecController = new XSecController(rxCtx);
-    mxSecurityController = mpXSecController;
     mbError = false;
 }