#4837 bind8: uninitialized value flood in miniserv.error if no views exist

[Bill Cole]

It looks like each domain get one of these in the miniserv.error log:

Use of uninitialized value in string eq at /usr/local/lib/webmin-1.820/bind8/index.cgi line 456.

and one of these:

Use of uninitialized value in string eq at ./bind8-lib.pl line 915.

every time the index of the bind8 module is loaded with no views defined for the server.

In addition, two BIND version checks use the wrong type of comparison:

Argument "9.9.2" isn't numeric in numeric ge (>=) at /usr/local/lib/webmin-1.820/bind8/index.cgi line 121.
Argument "9.9.2" isn't numeric in numeric ge (>=) at /usr/local/lib/webmin-1.820/bind8/index.cgi line 450.

This all seems to be cosmetic, apparently not causing any breakage but merely making that log very big and noisy, especially on a server with a large number of zones.

The attached patch addresses the uninitialized value messages by conditionally skipping over the tests generating them and corrects those two instances of using a numeric operator on a string.

[Jamie Cameron]

Thanks - these will all be fixed in the next Webmin release.