Packets: Fix VariantParser invalid parser access bug
[senf.git] / Packets / PacketData.hh
index 8a64ec0..7760200 100644 (file)
@@ -1,6 +1,8 @@
-// Copyright (C) 2007 
-// Fraunhofer Institut fuer offene Kommunikationssysteme (FOKUS)
-// Kompetenzzentrum fuer Satelitenkommunikation (SatCom)
+// $Id$
+//
+// Copyright (C) 2007
+// 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
 /** \file
     \brief PacketData public header */
 
-#ifndef HH_PacketData_
-#define HH_PacketData_ 1
+#ifndef HH_SENF_Packets_PacketData_
+#define HH_SENF_Packets_PacketData_ 1
 
 // Custom includes
 #include <boost/utility.hpp>
 #include <boost/type_traits.hpp>
-#include <boost/iterator/iterator_facade.hpp>
-#include "Utils/SafeBool.hh"
+#include "../Utils/safe_bool.hh"
+#include "../Utils/Exception.hh"
 #include "PacketTypes.hh"
 
 //#include "PacketData.mpp"
@@ -95,12 +97,19 @@ namespace senf {
         ///\name Sequence interface to raw data
         ///@{
 
-        iterator begin() const;
-        iterator end() const;
-        size_type size() const;
-        bool empty() const;
-        byte operator[](size_type n) const;
-        byte & operator[](size_type n);
+        iterator begin() const; ///< Return iterator to beginning
+                                /**< Returns an <em>random access iterator</em> referring to the
+                                     first byte of the packet data. */
+        iterator end() const; ///< Return iterator to end 
+                              /**< Returns an <em>random access iterator</em> referring to the 
+                                   byte past the end of the packet data. */
+        size_type size() const; ///< Returns the number of bytes in the packet data.
+        bool empty() const; ///< Test whether the packet data is empty.
+                            /**< Returns whether the packet data is empty, i.e. whether its size
+                                 is 0. This function does not modify the content of the packet 
+                                 data in any way. To clear the content use clear() */        
+        byte operator[](size_type n) const; ///< Access byte in the packet data
+        byte & operator[](size_type n); ///< Access byte in the packet data
 
         // Modifying the raw packet data
 
@@ -120,7 +129,7 @@ namespace senf {
 
         void erase(iterator pos);
         void erase(iterator first, iterator last);
-        void clear();
+        void clear(); ///< All bytes of the packet data dropped, leaving the container with a size of 0. */
         
         void resize(size_type n, byte v=0);
 
@@ -150,76 +159,15 @@ namespace senf {
         This exception is signaled whenever an operation tries to access an out-of-bounds data
         byte. If the packet has been implemented correctly, this signals a malformed packet.
      */
-    struct TruncatedPacketException : public std::exception
-    { virtual char const * what() const throw() { return "truncated packet"; } };
-
-    /** \brief Re-validating data iterator
-
-        This class is a wrapper around a PacketData::iterator instance. It will revalidate the
-        iterator on every access. This keeps the iterator valid even when the data container is
-        resized and thereby possibly relocated. The iterator will always point to the byte at the
-        same offset from the packets beginning. If data is inserted before this iterators position,
-        the data pointed to will of course change.
-
-        For this to work, the safe_data_iterator must be initialized with the container to which the
-        iterator belongs. After this initialization it can be used like any other iterator.
-     */
-    class safe_data_iterator
-        : public boost::iterator_facade< safe_data_iterator,
-                                         PacketData::value_type,
-                                         boost::random_access_traversal_tag >,
-          public ComparableSafeBool<safe_data_iterator>
-    {
-    public:
-        typedef PacketData::size_type size_type;
-
-        safe_data_iterator(); ///< Make uninitialized iterator
-        explicit safe_data_iterator(PacketData & data); 
-                                        ///< Construct iterator only setting the data container
-        safe_data_iterator(PacketData & data, PacketData::iterator i);
-                                        ///< Initialize iterator to given position
-        explicit safe_data_iterator(PacketParserBase const & parser);
-                                        ///< Initialize iterator from parser
-                                        /**< The iterator will point to the parsers start
-                                             position. */
-
-        safe_data_iterator & operator=(PacketData::iterator i); ///< Assign iterator
-                                        /**< The iteator \a i must be from the container wo which \c
-                                             this iterator has been initialized. */
-        safe_data_iterator & operator=(PacketParserBase const & parser);
-                                        ///< Assign iterator from parser
-                                        /**< The iterator will point to the parser start
-                                             position. */
+    struct TruncatedPacketException : public senf::Exception
+    { TruncatedPacketException() : senf::Exception("truncated packet"){} };
 
-        operator PacketData::iterator() const; ///< Convert to iterator
-
-        bool boolean_test() const;      ///< Check, if iterator is initialized
-
-        PacketData & data() const;      ///< Access data container
-
-    private:
-        friend class boost::iterator_core_access;
-
-        // iterator_facade interface
-
-        value_type & dereference() const;
-        bool equal(safe_data_iterator const & other) const;
-        difference_type distance_to(safe_data_iterator const & other) const;
-        void increment();
-        void decrement();
-        void advance(difference_type n);
-
-        PacketData::iterator i() const;
-
-        PacketData * data_;
-        size_type i_;
-    };
 }
 
 ///////////////////////////////hh.e////////////////////////////////////////
 #endif
-#if !defined(SENF_PACKETS_DECL_ONLY) && !defined(HH_PacketData_i_)
-#define HH_PacketData_i_
+#if !defined(HH_SENF_Packets_Packets__decls_) && !defined(HH_SENF_Packets_PacketData_i_)
+#define HH_SENF_Packets_PacketData_i_
 #include "PacketData.cci"
 //#include "PacketData.ct"
 #include "PacketData.cti"