From afe807c61372fe2481e73af63c8382af1e1d3011 Mon Sep 17 00:00:00 2001
From: Mauro Sardara <msardara@cisco.com>
Date: Wed, 4 Mar 2020 16:21:54 +0100
Subject: [HICN-540] Optimizations for libhicntransport

Change-Id: I8b46b4eb2ef5488c09041887cc8296a216440f33
Signed-off-by: Mauro Sardara <msardara@cisco.com>
---
 libtransport/includes/hicn/transport/core/packet.h | 72 ++++++++++++++++++++--
 1 file changed, 67 insertions(+), 5 deletions(-)

(limited to 'libtransport/includes/hicn/transport/core/packet.h')

diff --git a/libtransport/includes/hicn/transport/core/packet.h b/libtransport/includes/hicn/transport/core/packet.h
index e758aa13e..3ddc4a595 100644
--- a/libtransport/includes/hicn/transport/core/packet.h
+++ b/libtransport/includes/hicn/transport/core/packet.h
@@ -17,6 +17,7 @@
 
 #include <hicn/transport/core/name.h>
 #include <hicn/transport/core/payload_type.h>
+#include <hicn/transport/errors/malformed_packet_exception.h>
 #include <hicn/transport/portability/portability.h>
 #include <hicn/transport/security/crypto_hasher.h>
 #include <hicn/transport/security/crypto_suite.h>
@@ -81,7 +82,12 @@ class Packet : public std::enable_shared_from_this<Packet> {
   virtual ~Packet();
 
   static std::size_t getHeaderSizeFromFormat(Format format,
-                                             std::size_t signature_size = 0);
+                                             std::size_t signature_size = 0) {
+    std::size_t header_length;
+    hicn_packet_get_header_length_from_format(format, &header_length);
+    int is_ah = _is_ah(format);
+    return is_ah * (header_length + signature_size) + (!is_ah) * header_length;
+  }
 
   static std::size_t getHeaderSizeFromBuffer(Format format,
                                              const uint8_t *buffer);
@@ -91,9 +97,26 @@ class Packet : public std::enable_shared_from_this<Packet> {
 
   static bool isInterest(const uint8_t *buffer);
 
-  static Format getFormatFromBuffer(const uint8_t *buffer);
+  static Format getFormatFromBuffer(const uint8_t *buffer) {
+    Format format = HF_UNSPEC;
 
-  virtual void replace(MemBufPtr &&buffer);
+    if (TRANSPORT_EXPECT_FALSE(
+            hicn_packet_get_format((const hicn_header_t *)buffer, &format) <
+            0)) {
+      throw errors::MalformedPacketException();
+    }
+
+    return format;
+  }
+
+  TRANSPORT_ALWAYS_INLINE void replace(MemBufPtr &&buffer) {
+    packet_ = std::move(buffer);
+    packet_start_ = reinterpret_cast<hicn_header_t *>(packet_->writableData());
+    header_head_ = packet_.get();
+    payload_head_ = nullptr;
+    format_ = getFormatFromBuffer(reinterpret_cast<uint8_t *>(packet_start_));
+    name_.clear();
+  }
 
   std::size_t payloadSize() const;
 
@@ -123,6 +146,19 @@ class Packet : public std::enable_shared_from_this<Packet> {
 
   std::unique_ptr<utils::MemBuf> getPayload() const;
 
+  std::pair<const uint8_t *, std::size_t> getPayloadReference() const {
+    int signature_size = 0;
+    if (_is_ah(format_)) {
+      signature_size = (uint32_t)getSignatureSize();
+    }
+
+    auto header_size = getHeaderSizeFromFormat(format_, signature_size);
+    auto payload_length = packet_->length() - header_size;
+
+    return std::make_pair(packet_->data() + header_size,
+                          payload_length);
+  }
+
   Packet &updateLength(std::size_t length = 0);
 
   PayloadType getPayloadType() const;
@@ -152,7 +188,21 @@ class Packet : public std::enable_shared_from_this<Packet> {
   virtual utils::CryptoHash computeDigest(
       utils::CryptoHashType algorithm) const;
 
-  void setChecksum();
+  void setChecksum() {
+    uint16_t partial_csum = 0;
+
+    for (utils::MemBuf *current = header_head_->next();
+         current && current != header_head_; current = current->next()) {
+      if (partial_csum != 0) {
+        partial_csum = ~partial_csum;
+      }
+      partial_csum = csum(current->data(), current->length(), partial_csum);
+    }
+    if (hicn_packet_compute_header_checksum(format_, packet_start_,
+                                            partial_csum) < 0) {
+      throw errors::MalformedPacketException();
+    }
+  }
 
   bool checkIntegrity() const;
 
@@ -184,7 +234,19 @@ class Packet : public std::enable_shared_from_this<Packet> {
  private:
   virtual void resetForHash() = 0;
   void setSignatureSize(std::size_t size_bytes);
-  std::size_t getSignatureSize() const;
+
+  std::size_t getSignatureSize() const {
+    size_t size_bytes;
+    int ret =
+        hicn_packet_get_signature_size(format_, packet_start_, &size_bytes);
+
+    if (ret < 0) {
+      throw errors::RuntimeException("Packet without Authentication Header.");
+    }
+
+    return size_bytes;
+  }
+
   uint8_t *getSignature() const;
   void separateHeaderPayload();
 
-- 
cgit 1.2.3-korg