<html> <head> <meta content="text/html; charset=ISO-8859-1" http-equiv="Content-Type"> </head> <body bgcolor="#FFFFFF" text="#000000"> I just have one suggestion. Your code does not have default algorithm and key size if caller doesn't supply. How about default to, say, AES 128 if not supplied. Christina <div class="moz-cite-prefix">On 02/04/2014 10:30 AM, Ade Lee wrote: </div> <blockquote cite="mid:1391538651.25736.38.camel@aleeredhat.laptop" type="cite"> <pre wrap=""> Fix DRM archival, recovery and generation for non-DES3 keys. In the archival, recovery and generation code for symmetric keys, we use functions that require knowledge of the symmetric keys algorithm and key size. These were hardcoded to DES3, and so only DES3 worked. We added those parameters to the archival request, save them in the KeyRecord and retrive them when recovering the key. Tests have been added to DRMTest for the relevant usages. Thanks to jmagne for doing initial investigation and determining what needed to be done. Please review, Ade </pre> <fieldset class="mimeAttachmentHeader"></fieldset> <pre wrap="">_______________________________________________ Pki-devel mailing list <a class="moz-txt-link-abbreviated" href="mailto:Pki-devel@redhat.com">Pki-devel@redhat.com</a> <a class="moz-txt-link-freetext" href="https://www.redhat.com/mailman/listinfo/pki-devel">https://www.redhat.com/mailman/listinfo/pki-devel</a></pre> </blockquote> </body> </html>