Dereferencing freed pointer

[Steven Lougheed]

In function mqtt3_db_message_store in database.c the 'temp' pointer is freed and then dereferenced in a subsequent call to _mosquitto_free().

Line 464: _mosquitto_free(temp);
Line 465: _mosquitto_free(temp->source_id);

This seems bad.

[Roger Light (roger.light)]

It does seem bad. I've fixed it in this changeset: https://bitbucket.org/oojah/mosquitto/changeset/abe0ccd67aaf

Thanks for reporting it.

[Steven Lougheed (slougheed)]

Thanks for creating a great project and for the remarkably fast response times.