Increase buffer construction requirements (#88)

* Fix crash in Buffer with null pointer and non-zero length

* Throw inside Buffer constructor instead

Author: accelerated

include/cppkafka/buffer.h

@@ -35,6 +35,7 @@
 #include <iosfwd>
 #include <algorithm>
 #include "macros.h"
+#include "exceptions.h"
 
 namespace cppkafka {
 
@@ -75,6 +76,9 @@ class CPPKAFKA_API Buffer {
     Buffer(const T* data, size_t size) 
     : data_(reinterpret_cast<const DataType*>(data)), size_(size) {
         static_assert(sizeof(T) == sizeof(DataType), "sizeof(T) != sizeof(DataType)");
+        if ((data_ == nullptr) && (size_ > 0)) {
+            throw Exception("Invalid buffer configuration");
+        }
     }
 
     /**

tests/buffer_test.cpp

@@ -15,6 +15,9 @@ TEST_CASE("conversions", "[buffer]") {
     const Buffer buffer(data);
     const Buffer empty_buffer;
 
+    SECTION("construction") {
+        CHECK_THROWS_AS(Buffer((const char*)nullptr, 5), Exception);
+    }
 
     SECTION("bool conversion") {
         CHECK(!!buffer == true);