From 89619dc379185323c12c0be27ca9a0abf58b8d84 Mon Sep 17 00:00:00 2001
From: Jean-Marie Renouard <jmrenouard@gmail.com>
Date: Sat, 17 Jun 2023 18:11:00 +0200
Subject: [PATCH] skip_name_resolve not interpreted properly #675

---
 mysqltuner.pl | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/mysqltuner.pl b/mysqltuner.pl
index 8562704..c6758bc 100644
--- a/mysqltuner.pl
+++ b/mysqltuner.pl
@@ -3305,6 +3305,7 @@ sub mysql_stats {
     }
 
     # name resolution
+    debugprint "skip name resolve: $result{'Variables'}{'skip_name_resolve'}" if ( defined( $result{'Variables'}{'skip_name_resolve'} ) );
     if ( defined( $result{'Variables'}{'skip_networking'} )
         && $result{'Variables'}{'skip_networking'} eq 'ON' )
     {
@@ -3329,8 +3330,8 @@ sub mysql_stats {
             push( @adjvars, "skip-name-resolve=0" );
         }
     }
-    elsif ( $result{'Variables'}{'skip_name_resolve'} ne 'OFF'
-        and $result{'Variables'}{'skip_name_resolve'} ne '0' )
+    elsif ( $result{'Variables'}{'skip_name_resolve'} ne 'ON'
+        and $result{'Variables'}{'skip_name_resolve'} ne '1' )
     {
         badprint
 "Name resolution is active: a reverse name resolution is made for each new connection which can reduce performance";