Hide Algorithm_Factory and use the functions in lookup.h internally.

Fix two memory leaks (in TLS and modes) caused by calling get_foo and
then cloning the result before saving it (leaking the original object),
a holdover from the conversion between construction techniques in 1.11.14

1 files changed, 4 insertions, 1 deletions

diff --git a/src/lib/mac/cmac/cmac.cpp b/src/lib/mac/cmac/cmac.cpp
index a77673138..0b867af8f 100644
--- a/src/lib/mac/cmac/cmac.cpp
+++ b/src/lib/mac/cmac/cmac.cpp
@@ -13,7 +13,10 @@ namespace Botan {
 CMAC* CMAC::make(const Spec& spec)
    {
    if(spec.arg_count() == 1)
-      return new CMAC(Algo_Registry<BlockCipher>::global_registry().make(spec.arg(0)));
+      {
+      if(BlockCipher* bc = get_block_cipher(spec.arg(0)))
+         return new CMAC(bc);
+      }
    return nullptr;
    }