Packets: Fix VariantParser invalid parser access bug

[senf.git] / Packets / PacketRegistry.ih

diff --git a/Packets/PacketRegistry.ih b/Packets/PacketRegistry.ih
index 94b9169..af16278 100644 (file)
--- a/Packets/PacketRegistry.ih
+++ b/Packets/PacketRegistry.ih
@@ -1,9 +1,9 @@
 // $Id$
 //
 // Copyright (C) 2006
-// Fraunhofer Institut fuer offene Kommunikationssysteme (FOKUS)
-// Kompetenzzentrum fuer Satelitenkommunikation (SatCom)
-//     Stefan Bund <stefan.bund@fokus.fraunhofer.de>
+// Fraunhofer Institute for Open Communication Systems (FOKUS)
+// Competence Center NETwork research (NET), St. Augustin, GERMANY
+//     Stefan Bund <g0dil@berlios.de>
 //
 // 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
@@ -20,46 +20,98 @@
 // Free Software Foundation, Inc.,
 // 59 Temple Place - Suite 330, Boston, MA  02111-1307, USA.
 
-#ifndef IH_PacketRegistryImpl_
-#define IH_PacketRegistryImpl_ 1
+/** \file
+    \brief PacketRegistry internal header */
+
+#ifndef IH_SENF_Packets_PacketRegistry_
+#define IH_SENF_Packets_PacketRegistry_ 1
 
 // Custom includes
+#include <ext/functional>
 #include <boost/intrusive_ptr.hpp>
-#include "typeidvalue.hh"
+#include <boost/iterator/transform_iterator.hpp>
+#include "../Utils/TypeIdValue.hh"
 
 ///////////////////////////////ih.p////////////////////////////////////////
 
 namespace senf {
+
+    /** \brief Registry entry
+
+        Value returned by a registry lookup
+     */
+    struct PkReg_Entry
+        : public intrusive_refcount
+    {
+        virtual ~PkReg_Entry();
+        virtual Packet::factory_t factory() const = 0;
+                                        ///< Get factory of the registered packet type
+        virtual std::string name() const = 0;
+    };
+
 namespace detail {
 
+    /** \brief Internal: Registry entry implementation for a specific packet type
+
+        \internal
+     */
     template <class PacketType>
     struct PkReg_EntryImpl
         : public PkReg_Entry
     {
-        virtual PacketInterpreterBase::factory_t factory() const;
+        virtual Packet::factory_t factory() const;
+        virtual std::string name() const;
     };
 
-    template <class KeyType>
-    class PacketRegistryImpl 
+    /** \brief Internal: Registry implementation base-class and registry of registries
+
+        \internal
+     */
+    class PacketRegistryImplBase
         : private boost::noncopyable
     {
     public:
-        ///////////////////////////////////////////////////////////////////////////
-        // Types
+        virtual ~PacketRegistryImplBase();
 
-        typedef KeyType key_t;
+        static void dump(std::ostream & os);
+
+    protected:
+        typedef std::map<std::string, PacketRegistryImplBase*>  RegistryMap;
+        static RegistryMap & registries();
+
+    private:
+        virtual void v_dump(std::ostream & os) = 0;
+    };
+
+    /** \brief Internal: Singleton class implementing the packet registry.
 
+        \internal
+     */
+    template <class KeyType>
+    class PacketRegistryImpl
+        : public PacketRegistryImplBase
+    {
+    public:
+        typedef KeyType key_t;
         typedef PkReg_Entry Entry;
 
+    private:
+        typedef boost::intrusive_ptr<Entry> Entry_ptr;
+        typedef std::map<key_t, Entry_ptr> PacketMap;
+        typedef std::map<senf::TypeIdValue, key_t> ReversePacketMap;
+
+    public:
+        ///////////////////////////////////////////////////////////////////////////
+        // Types
+
+        typedef boost::transform_iterator< ::__gnu_cxx::select1st<typename PacketMap::value_type>,
+                                           typename PacketMap::const_iterator > iterator;
+
         ///////////////////////////////////////////////////////////////////////////
         ///\name Structors and default members
         ///@{
 
-        // default default constructor
-        // no copy constructor
-        // no copy assignment
-        // default destructor
-        // no conversion constructors
+        PacketRegistryImpl(std::string const & name);
 
         ///@}
         ///////////////////////////////////////////////////////////////////////////
@@ -73,12 +125,13 @@ namespace detail {
         Entry const & lookup(key_t key);
         Entry const * lookup(key_t key, bool);
 
+        iterator begin() const;
+        iterator end() const;
+
     protected:
 
     private:
-        typedef boost::intrusive_ptr<Entry> Entry_ptr;
-        typedef std::map<key_t, Entry_ptr> PacketMap;
-        typedef std::map<senf::TypeIdValue, key_t> ReversePacketMap;
+        virtual void v_dump(std::ostream & os);
 
         PacketMap registry_;
         ReversePacketMap reverseRegistry_;