Return code unchecked during publish

[Steven Lougheed]

In _mosquitto_handle_publish (read_handle.c line 120), the return code from _mosquitto_read_bytes is not saved. The next line attempts to look at the return code and cleanup if the call fails, but this condition can never be met.

OLD:
_mosquitto_read_bytes(&mosq->in_packet, message->msg.payload, message->msg.payloadlen);
NEW
rc = _mosquitto_read_bytes(&mosq->in_packet, message->msg.payload, message->msg.payloadlen);

[Roger Light (roger.light)]

Thanks for the report, and well spotted. I've commited the change: https://bitbucket.org/oojah/mosquitto/changeset/42b40d98395a