Fix crash when new topic is not created.

[Mostafa Razavi (elektito)]

I found that the following code caused a crash:

from confluent_kafka.admin import AdminClient
from confluent_kafka.cimpl import NewTopic

client = AdminClient({})
topics = [NewTopic(topic='foo', num_partitions=0)]
client.create_topics(topics)

This pull request, fixes the issue. The problem was that when a topic failed to be created, like in this case where the number of partitions is zero, we would still try to free the topic, causing a null pointer to be de-referenced in rdkafka.

While, this is also a bug in rdkafka, since a null pointer should not be de-referenced either way, we should not pass such a pointer to a library either.

[Deleted user (ghost)]

It looks like Mostafa Razavi (@elektito) hasn't signed our Contributor License Agreement, yet.

The purpose of a CLA is to ensure that the guardian of a project's outputs has the necessary ownership or grants of rights over all contributions to allow them to distribute under the chosen licence.
Wikipedia

You can read and sign our full Contributor License Agreement here.

Once you've signed reply with [clabot:check] to prove it.

Appreciation of efforts,

clabot

[Mostafa Razavi (elektito)]

[clabot:check]

[Deleted user (ghost)]

Confluent Inc. (@confluentinc) It looks like Mostafa Razavi (@elektito) just signed our Contributor License Agreement. 👍

Always at your service,

clabot

[Magnus Edenhill (edenhill)]

Good fix, thank you!