If statement for detecting if a pointer is NULL
is totally unnecessary.

Signed-off-by: YangHau <vulxj0j8j8@gmail.com>

Closes #1369. Thanks to TimmvonderMehden

These were being treated as a protocol error.

Closes #1367. Thanks to Frank Pagliughi.

Thanks to Greg Troxel.

feat: Remove unecessary if statement

free() would take no action if the arguement is a NULL pointer.
Therefore, the if statement is totally unnecessary.
See ISO-IEC 9899

Signed-off-by: YangHau <vulxj0j8j8@gmail.com>

Closes #1344. Thanks to HowJMay.

Closes #1361. Thanks to Duck Development.

Closes #1362. Thanks to Furkan Aksoy.

Fix password input when MAX_BUFFER_LEN is not 1024 in mosquitto_passwd.c

Remove redundant initialization in clients

docs: Fix typo on man-page for mosquitto.conf

- man-page has a typo in the 'bridge_capath' section under SSL/TLS support

Signed-off-by: Jonas Helgemo <jhel@pm.me>

Fix incoming messages not being removed for a client if the topic being
published to does not have any subscribers.

Closes #1322. Thanks to Yannic Schröder.

Closes #1332. Thanks to xmas79.

`mosquitto_string_option(mosq, MOSQ_OPT_TLS_KEYFORM, ...)` was incorrectly
returning `MOSQ_ERR_INVAL` with valid input. This has been fixed.

Closes #1360. Thanks to Michael Dombrowski.

`mosquitto_connect_async()` is now consistent with `mosquitto_connect()`
when connecting to a non-existent server.

Closes #1345. Thanks to Mohammad Reza.

* Fix typo

Closes #1354. Thanks to Ben Barbour.

Closes #1353. Thanks to Ben Barbour.

the `memset(&cfg, 0, sizeof(struct mosq_config));` already exsits in
`client_config_load()`'s `init_config()` function call.
So calling it in main function is totally unnecessary.

Signed-off-by: YangHau <vulxj0j8j8@gmail.com>

This change means that users with the appropriate security permissions
can open the log file for reading at the same time that it is being
written.

Closes #515.

Closes #1272. Thanks to BowenMarmot and addendumE.

No pthread_cancel() on Android