Forbid copying an Algorithm_Factory; could easily cause double-delete,

especially in a multithreaded environment, and doesn't seem like a
useful operation to support.

(In principle, we could support this by adding a clone() call to
Algorithm_Cache, which would in turn call clone on each of it's held
prototype objects, plus adding a clone to Engine. Doesn't seem worth the
bother, though.

1 files changed, 4 insertions, 0 deletions

diff --git a/src/algo_factory/algo_factory.h b/src/algo_factory/algo_factory.h
index f7e85202b..a35a76de3 100644
--- a/src/algo_factory/algo_factory.h
+++ b/src/algo_factory/algo_factory.h
@@ -183,6 +183,10 @@ class BOTAN_DLL Algorithm_Factory
       friend class Engine_Iterator;
 
    private:
+      Algorithm_Factory(const Algorithm_Factory&) {}
+      Algorithm_Factory& operator=(const Algorithm_Factory&)
+         { return (*this); }
+
       class Engine* get_engine_n(u32bit) const;
 
       std::vector<class Engine*> engines;