search

nicolasff / webdis

A Redis HTTP interface with JSON output
https://webd.is
BSD 2-Clause "Simplified" License
2.84k stars 304 forks source link

Fix small leaks in conf.c (fixes #184) #185

Closed jessie-murray closed 3 years ago

jessie-murray commented 3 years ago
  1. plaintext was not free'd after encoding credentials
  2. ACL commands were duplicated when there was no need to

In both cases the value came from conf_string_or_envvar which always uses strdup.

nicolasff commented 3 years ago

Looks good! Thanks again @jessie-murray.