Comment / tidy.
[libdcp.git] / src / signer.cc
index 999ee59465e7874d7b22bb328844f9fade6958e6..a280cb07af0ff693fd15f306fca1dbe9ce31584a 100644 (file)
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2013 Carl Hetherington <cth@carlh.net>
+    Copyright (C) 2013-2014 Carl Hetherington <cth@carlh.net>
 
     This program is free software; you can redistribute it and/or modify
     it under the terms of the GNU General Public License as published by
 
 */
 
+/** @file  src/signer.cc
+ *  @brief Signer class.
+ */
+
+#include "signer.h"
+#include "exceptions.h"
+#include <libcxml/cxml.h>
 #include <libxml++/libxml++.h>
 #include <xmlsec/xmldsig.h>
 #include <xmlsec/dl.h>
 #include <xmlsec/app.h>
-#include "signer.h"
-#include "exceptions.h"
+#include <xmlsec/crypto.h>
 
 using std::string;
 using std::list;
+using std::cout;
 using boost::shared_ptr;
-using namespace libdcp;
+using namespace dcp;
 
-/** @param signer_key Filename of private key to sign with */
+/** Add a &lt;Signer&gt; and &lt;ds:Signature&gt; nodes to an XML node.
+ *  @param parent XML node to add to.
+ *  @param standard INTEROP or SMPTE.
+ */
 void
-Signer::sign (xmlpp::Element* parent, bool interop) const
+Signer::sign (xmlpp::Element* parent, Standard standard) const
 {
-       add_signer (parent, "dsig");
+       /* <Signer> */
+       
+       xmlpp::Element* signer = parent->add_child("Signer");
+       xmlpp::Element* data = signer->add_child("X509Data", "dsig");
+       xmlpp::Element* serial_element = data->add_child("X509IssuerSerial", "dsig");
+       serial_element->add_child("X509IssuerName", "dsig")->add_child_text (_certificates.leaf()->issuer());
+       serial_element->add_child("X509SerialNumber", "dsig")->add_child_text (_certificates.leaf()->serial());
+       data->add_child("X509SubjectName", "dsig")->add_child_text (_certificates.leaf()->subject());
 
+       /* <Signature> */
+       
        xmlpp::Element* signature = parent->add_child("Signature", "dsig");
        
-       {
-               xmlpp::Element* signed_info = signature->add_child ("SignedInfo", "dsig");
-               signed_info->add_child("CanonicalizationMethod", "dsig")->set_attribute ("Algorithm", "http://www.w3.org/TR/2001/REC-xml-c14n-20010315");
-
-               if (interop) {
-                       signed_info->add_child("SignatureMethod", "dsig")->set_attribute("Algorithm", "http://www.w3.org/2000/09/xmldsig#rsa-sha1");
-               } else {
-                       signed_info->add_child("SignatureMethod", "dsig")->set_attribute("Algorithm", "http://www.w3.org/2001/04/xmldsig-more#rsa-sha256");
-               }
-               
-               {
-                       xmlpp::Element* reference = signed_info->add_child("Reference", "dsig");
-                       reference->set_attribute ("URI", "");
-                       {
-                               xmlpp::Element* transforms = reference->add_child("Transforms", "dsig");
-                               transforms->add_child("Transform", "dsig")->set_attribute (
-                                       "Algorithm", "http://www.w3.org/2000/09/xmldsig#enveloped-signature"
-                                       );
-                       }
-                       reference->add_child("DigestMethod", "dsig")->set_attribute("Algorithm", "http://www.w3.org/2000/09/xmldsig#sha1");
-                       /* This will be filled in by the signing later */
-                       reference->add_child("DigestValue", "dsig");
-               }
+       xmlpp::Element* signed_info = signature->add_child ("SignedInfo", "dsig");
+       signed_info->add_child("CanonicalizationMethod", "dsig")->set_attribute ("Algorithm", "http://www.w3.org/TR/2001/REC-xml-c14n-20010315");
+       
+       if (standard == INTEROP) {
+               signed_info->add_child("SignatureMethod", "dsig")->set_attribute("Algorithm", "http://www.w3.org/2000/09/xmldsig#rsa-sha1");
+       } else {
+               signed_info->add_child("SignatureMethod", "dsig")->set_attribute("Algorithm", "http://www.w3.org/2001/04/xmldsig-more#rsa-sha256");
        }
        
+       xmlpp::Element* reference = signed_info->add_child("Reference", "dsig");
+       reference->set_attribute ("URI", "");
+
+       xmlpp::Element* transforms = reference->add_child("Transforms", "dsig");
+       transforms->add_child("Transform", "dsig")->set_attribute (
+               "Algorithm", "http://www.w3.org/2000/09/xmldsig#enveloped-signature"
+               );
+
+       reference->add_child("DigestMethod", "dsig")->set_attribute("Algorithm", "http://www.w3.org/2000/09/xmldsig#sha1");
+       /* This will be filled in by the signing later */
+       reference->add_child("DigestValue", "dsig");
+
+       signature->add_child("SignatureValue", "dsig");
+       signature->add_child("KeyInfo", "dsig");
        add_signature_value (signature, "dsig");
 }
 
 
-/** Sign an XML node.  This function takes a certificate chain (to prove that the sender is bona fide) and
- *  a private key with which to sign the node.
+/** Sign an XML node.
  *
  *  @param parent Node to sign.
- *  @param certificates Certificate chain for the signer.
- *  @param signer_key Filename of the private key of the signer.
  *  @param ns Namespace to use for the signature XML nodes.
  */
 void
-Signer::add_signature_value (xmlpp::Element* parent, string ns) const
+Signer::add_signature_value (xmlpp::Node* parent, string ns) const
 {
-       parent->add_child("SignatureValue", ns);
+       cxml::Node cp (parent);
+       xmlpp::Node* key_info = cp.node_child("KeyInfo")->node ();
 
-       /* Add the certificate chain to a KeyInfo child node of parent */
-       xmlpp::Element* key_info = parent->add_child("KeyInfo", ns);
+       /* Add the certificate chain to the KeyInfo child node of parent */
        list<shared_ptr<Certificate> > c = _certificates.leaf_to_root ();
        for (list<shared_ptr<Certificate> >::iterator i = c.begin(); i != c.end(); ++i) {
                xmlpp::Element* data = key_info->add_child("X509Data", ns);
@@ -99,9 +113,9 @@ Signer::add_signature_value (xmlpp::Element* parent, string ns) const
                throw MiscError ("could not create signature context");
        }
 
-       signature_context->signKey = xmlSecCryptoAppKeyLoad (_key.c_str(), xmlSecKeyDataFormatPem, 0, 0, 0);
+       signature_context->signKey = xmlSecCryptoAppKeyLoad (_key.string().c_str(), xmlSecKeyDataFormatPem, 0, 0, 0);
        if (signature_context->signKey == 0) {
-               throw FileError ("could not load private key file", _key);
+               throw FileError ("could not load private key file", _key, 0);
        }
 
        /* XXX: set key name to the file name: is this right? */
@@ -115,21 +129,3 @@ Signer::add_signature_value (xmlpp::Element* parent, string ns) const
 
        xmlSecDSigCtxDestroy (signature_context);
 }
-
-void
-Signer::add_signer (xmlpp::Element* parent, string ns) const
-{
-       xmlpp::Element* signer = parent->add_child("Signer");
-
-       {
-               xmlpp::Element* data = signer->add_child("X509Data", ns);
-               
-               {
-                       xmlpp::Element* serial_element = data->add_child("X509IssuerSerial", ns);
-                       serial_element->add_child("X509IssuerName", ns)->add_child_text (_certificates.leaf()->issuer());
-                       serial_element->add_child("X509SerialNumber", ns)->add_child_text (_certificates.leaf()->serial());
-               }
-               
-               data->add_child("X509SubjectName", ns)->add_child_text (_certificates.leaf()->subject());
-       }
-}