Add some improvements

Author: Hind-M

src/serialize.cpp

@@ -193,11 +193,19 @@ std::vector<uint8_t> serialize_primitive_array(const sparrow::primitive_array<T>
 
         // Write the 4-byte metadata length for the RecordBatch message
         memcpy(dst, &batch_meta_len, sizeof(batch_meta_len));
-        dst += sizeof(uint32_t);
+        dst = dst + sizeof(uint32_t);
         // Copy the RecordBatch metadata into the buffer
         memcpy(dst, batch_builder.GetBufferPointer(), batch_meta_len);
         // Add padding to align the body to an 8-byte boundary
-        memset(dst + batch_meta_len, 0, aligned_batch_meta_len - batch_meta_len);
+        if (aligned_batch_meta_len >= batch_meta_len)
+        {
+            memset(dst + batch_meta_len, 0, aligned_batch_meta_len - batch_meta_len);
+        }
+        else
+        {
+            throw std::runtime_error("aligned_batch_meta_len should be greater than batch_meta_len");
+        }
+
         dst += aligned_batch_meta_len;
         // Copy the actual data buffers (the message body) into the buffer
         if (validity_bitmap)
@@ -271,10 +279,10 @@ sparrow::primitive_array<T> deserialize_primitive_array(const std::vector<uint8_
     int64_t validity_len = buffers_meta->Get(0)->length();
     int64_t data_len = buffers_meta->Get(1)->length();
 
-    uint8_t* validity_buffer_copy = new uint8_t[validity_len];
+    auto validity_buffer_copy = new uint8_t[validity_len];
     memcpy(validity_buffer_copy, body_ptr + buffers_meta->Get(0)->offset(), validity_len);
 
-    uint8_t* data_buffer_copy = new uint8_t[data_len];
+    auto data_buffer_copy = new uint8_t[data_len];
     memcpy(data_buffer_copy, body_ptr + buffers_meta->Get(1)->offset(), data_len);
 
     // Get name