Freshclam Database Init

How about changing /etc/init.d/freshclam to check if values are being set, before adding them to config file, at least for those which else produce errors when the config is parsed?

[ -n "$ExtraDatabase" ] && echo "ExtraDatabase $ExtraDatabase"
[ -n "$PrivateMirror" ] && echo "PrivateMirror $PrivateMirror"
[ -n "$ExcludeDatabase" ] && echo "ExcludeDatabase $ExcludeDatabase"
[ -n "$DatabaseCustomURL" ] && echo "DatabaseCustomURL $DatabaseCustomURL"

Chances that the relevant maintainers will see this by accident here are quite low, re-spinning this as a PR against the freshclam package on github (and pinging the maintainer there) has a much higher chance to be seen, discussed and potentially acted upon.

1 Like