[avahi] [patch] add aliases configuration setting to register CNAME records

Markus Rathgeb maggu2810 at googlemail.com
Wed Aug 1 04:40:36 PDT 2012


Just two comments:

avahi-core/entry.c line 454:
memory leak: If you return on error because avahi_record_new_full
failed, fq will not be freed.

avahi-daemon/cnames.c line 78:
I do not know what is the common usage in the avahi sources, but I
would prefer to add a default case to a switch statement.
You should also add a break on the last cases. It will prevent errors
if you add other cases later.


More information about the avahi mailing list