Add FIXME

Change-Id: I37e614f6dfa0778cb126a6de282921a15417af1c

1 files changed, 6 insertions, 0 deletions

diff --git a/xmlsecurity/inc/xmlsignaturehelper.hxx b/xmlsecurity/inc/xmlsignaturehelper.hxx
index 4ba359c0bd67..3ff3a9cfc385 100644
--- a/xmlsecurity/inc/xmlsignaturehelper.hxx
+++ b/xmlsecurity/inc/xmlsignaturehelper.hxx
@@ -91,6 +91,12 @@ 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;