In 8c852cb the CPL root attribute "dsig" was removed (to prevent
authorCarl Hetherington <cth@carlh.net>
Thu, 5 Sep 2019 00:51:11 +0000 (01:51 +0100)
committerCarl Hetherington <cth@carlh.net>
Mon, 30 Sep 2019 22:22:09 +0000 (00:22 +0200)
'XML root element can contain only one namespace' errors)

In a266fc2 [Sony digest validation fix] this was (for some
unknown reason) re-added, which makes the root element bug recur.

This commit removes that namespace again.

It's a shame there was no unit test to pick this regression up.

src/cpl.cc

index 3eeb102c196b28ac83af9736ec1c997800ddded9..166edc86ad2145bfd8cc0db9ee104365612a775a 100644 (file)
@@ -140,10 +140,6 @@ CPL::write_xml (boost::filesystem::path file, Standard standard, shared_ptr<cons
                root = doc.create_root_node ("CompositionPlaylist", cpl_smpte_ns);
        }
 
-       if (signer) {
-               root->set_namespace_declaration ("http://www.w3.org/2000/09/xmldsig#", "dsig");
-       }
-
        root->add_child("Id")->add_child_text ("urn:uuid:" + _id);
        root->add_child("AnnotationText")->add_child_text (_metadata.annotation_text);
        root->add_child("IssueDate")->add_child_text (_metadata.issue_date);