search

martin-ger / esp_mqtt

MQTT Broker/Bridge on the ESP8266
MIT License
295 stars 69 forks source link

Update mqtt_server.c #16

Closed miolion closed 6 years ago

miolion commented 6 years ago

mqtt_server.c.zip

This is a test pull request from a Github beginner to fix the 'MQTT: Out of mem' minor issue...

SJ

martin-ger commented 6 years ago

Just found your pull request - thanks. Think it would be better to store the empty string "" than to set the pointer to zero, right?

miolion commented 6 years ago

Hi, I am going to close this pull request due to another side effect that if a client publish a message with the default user name and password, "none" and "", the broker will pass the authentication by default.

mosquitto_pub -h 192.168.4.1 -t hello -u "none" -P "" -m message

SJ