fix: use strict MariaDB detection in install advisor (#273)

Refs #273

Signed-off-by: Thomas Vincent <thomasvincent@gmail.com>

Author: somethingwithproof

setup.php

@@ -966,10 +966,11 @@ function syslog_install_advisor($syslog_exists) {
 		$type = __('Install', 'syslog');
 	}
 
-	$database = db_fetch_row('SHOW GLOBAL VARIABLES LIKE "version"');
+	syslog_connect();
+	$database = syslog_db_fetch_row('SHOW GLOBAL VARIABLES LIKE "version"');
 
-	/* remove Aria as a storage enging if this is mysql */
-	if (stripos($database['Value'], 'mariadb') == false) {
+	/* remove Aria as a storage engine if this is mysql */
+	if (stripos($database['Value'], 'mariadb') === false) {
 		unset($fields_syslog_update['engine']['array']['aria']);
 	} else {
 		$fields_syslog_update['engine']['value'] = 'aria';
@@ -1632,4 +1633,3 @@ function syslog_utilities_list() {
 	</tr>
 	<?php
 }
-

tests/regression/issue270_mariadb_detection_strict_test.php

@@ -0,0 +1,23 @@
+<?php
+
+$setup = file_get_contents(dirname(__DIR__, 2) . '/setup.php');
+
+if ($setup === false) {
+	fwrite(STDERR, "Failed to load setup.php\n");
+	exit(1);
+}
+
+$legacyPattern = '/stripos\s*\(\s*\$database\s*\[\s*[\'"]{1}Value[\'"]{1}\s*\]\s*,\s*[\'"]{1}mariadb[\'"]{1}\s*\)\s*==\s*false/';
+$fixedPattern  = '/stripos\s*\(\s*\$database\s*\[\s*[\'"]{1}Value[\'"]{1}\s*\]\s*,\s*[\'"]{1}mariadb[\'"]{1}\s*\)\s*===\s*false/';
+
+if (preg_match($legacyPattern, $setup)) {
+	fwrite(STDERR, "Legacy loose MariaDB stripos comparison is still present.\n");
+	exit(1);
+}
+
+if (!preg_match($fixedPattern, $setup)) {
+	fwrite(STDERR, "Strict MariaDB stripos comparison is missing.\n");
+	exit(1);
+}
+
+echo "issue270_mariadb_detection_strict_test passed\n";