Reason: The memory allocated for the packet pointer at line 155 is not freed
before returning at line 188.

Fix: I inserted the mosquitto__free(packet) statement just before returning
at line 188.

Signed-off-by: Panagiotis Vasilikos <panagiotis.vasilikos@alexandra.dk>

Reason: In line 70, the memory allocation for the pointer reasons_codes may
result to a memory leak due to the many returns (e.g as the one in line 78)
occuring in the program's path until reaching the mosquitto__free at line 122.

Fix: I moved the memory allocation code block (lines 69-73) just before
the line 102. This is the first place the pointer reason_codes is used, while
the following mosquitto__free operators free the allocated memory correctly.

Signed-off-by: Panagiotis Vasilikos <panagiotis.vasilikos@alexandra.dk>

Previously, authentication checks would only take place if usernames
were defined in the password file.

Closes #1545. Thanks to Timothy Godfrey.

Closes #1539. Thanks to Leon Poon.

Closes #1525. Thanks to Christoph Krey.

Closes #1513.

In sub_client.c, call mosquitto_destroy() in cleanup label.

This is all now handled in session_expiry.c, through session expiry interval/time.

Closes #1494. Thanks to Christoph Krey.

Fixes: #1513 and frees resources when someone terminates
via SIGTERM or SIGINT.

Signed-off-by: Basavesh Shivakumar <basavesh.shivakumar@gmail.com>

Closes #1499. Thanks to Basavesh Shivakumar.

Signed-off-by: Basavesh Shivakumar <basavesh.shivakumar@gmail.com>

Blank lines are sent as empty messages.

Closes #1474. Thanks to majekw.

This would happen if the final line did not end with a new line.

Closes #1473. Thanks to majekw.

This can occurs after one client takes over a previous connection.

Closes #1489. Thanks to Bas Verhoeven.

Install pkg-config files into arch dependent locations also for CMake builds

This allow more libraries to be linked when compiling the clients with a
static libmosquitto, as required for e.g. openssl on some systems.

Closes #1371. Thanks to Fabrice Fontaine.

This seems to be required just on more modern Python versions.

Signed-off-by: Mario Vejlupek <mario@vejlupek.cz>

Stdin mode wouldn't work with normal compiler optimisation levels.

mosquitto_pub: split main loop.

Fix test on invalid null will topic value in connect packet

Fix test on invalid reserved bit value in connect packet

Add send of pubcomp in 02-subpub-qos2 script. To keep the broker session working while launching several tests on the same broker, the last packet of this transaction must be sent.

Fix way of sending packets in compliance tests. According to the documentation of python 3 socket::send method (https://docs.python.org/3/library/socket.html#socket.socket.send), the call to send must be retry until all data is sent while sending packet with a "large" amount of data.

Signed-off-by: Jerome Malinge <gromgromm@yahoo.com>

Signed-off-by: Jerome Malinge <gromgromm@yahoo.com>

To keep the broker session working while launching several tests on
the same broker, the last packet of this transaction must be sent.

Signed-off-by: Jerome Malinge <gromgromm@yahoo.com>

According to the documentation of python 3 socket::send method
(https://docs.python.org/3/library/socket.html#socket.socket.send

),
the call to send must be retry until all data is sent while sending
packet with a "large" amount of data.

Signed-off-by: Jerome Malinge <gromgromm@yahoo.com>

They would be be sent without an expiry interval property just before
they were expired.

Closes #1464. Thanks to Dustin Sallings.