Fix two potential leaks.

pull/2215/head
Roger Light 4 years ago
parent be96d21e7b
commit 9c9ca33d63

@ -236,7 +236,7 @@ int mosquitto_tls_opts_set(struct mosquitto *mosq, int cert_reqs, const char *tl
mosq->tls_13_ciphers = NULL; mosq->tls_13_ciphers = NULL;
if(ciphers){ if(ciphers){
if(!strcasecmp(tls_version, "tlsv1.3")){ if(!strcasecmp(mosq->tls_version, "tlsv1.3")){
mosq->tls_13_ciphers = mosquitto__strdup(ciphers); mosq->tls_13_ciphers = mosquitto__strdup(ciphers);
if(!mosq->tls_13_ciphers) return MOSQ_ERR_NOMEM; if(!mosq->tls_13_ciphers) return MOSQ_ERR_NOMEM;
}else{ }else{

@ -207,6 +207,7 @@ int mosquitto_broker_publish_copy(
mosquitto_property *properties) mosquitto_property *properties)
{ {
void *payload_out; void *payload_out;
int rc;
if(topic == NULL if(topic == NULL
|| payloadlen < 0 || payloadlen < 0
@ -222,7 +223,7 @@ int mosquitto_broker_publish_copy(
} }
memcpy(payload_out, payload, (size_t)payloadlen); memcpy(payload_out, payload, (size_t)payloadlen);
return mosquitto_broker_publish( rc = mosquitto_broker_publish(
clientid, clientid,
topic, topic,
payloadlen, payloadlen,
@ -230,6 +231,11 @@ int mosquitto_broker_publish_copy(
qos, qos,
retain, retain,
properties); properties);
if(rc){
free(payload_out);
}
return rc;
} }

Loading…
Cancel
Save