Merge pull request mfontanini#234 from accelerated/null_topic

mfontanini · web-flow · commit f1de729d4e2d · 2020-01-27T09:09:44.000-08:00
Fix crash when message handle is valid but topic is null
diff --git a/include/cppkafka/message.h b/include/cppkafka/message.h
@@ -108,7 +108,7 @@ class CPPKAFKA_API Message {
      */
     std::string get_topic() const {
         assert(handle_);
-        return rd_kafka_topic_name(handle_->rkt);
+        return handle_->rkt ? rd_kafka_topic_name(handle_->rkt) : std::string{};
     }
 
     /**

Original file line number	Diff line number	Diff line change
`@@ -108,7 +108,7 @@ class CPPKAFKA_API Message {`
`108`	`108`	`*/`
`109`	`109`	`std::string get_topic() const {`
`110`	`110`	`assert(handle_);`
`111`		`- return rd_kafka_topic_name(handle_->rkt);`
	`111`	`+ return handle_->rkt ? rd_kafka_topic_name(handle_->rkt) : std::string{};`
`112`	`112`	`}`
`113`	`113`
`114`	`114`	`/**`