#3356 fix for sp_validatelogins

Author: Montro1981

sp_Blitz.sql

@@ -297,16 +297,29 @@ AS
                 SET @SkipXPCMDShell = 1;
             END; /*Need execute on xp_cmdshell*/
 
-            IF NOT EXISTS
-            (
-                SELECT
-                    1/0
-                FROM fn_my_permissions(N'sp_validatelogins', N'OBJECT') AS fmp
-                WHERE fmp.permission_name = N'EXECUTE'
-            )
-            BEGIN
-                SET @SkipValidateLogins = 1;
-            END; /*Need execute on sp_validatelogins*/
+			IF ISNULL(@SkipValidateLogins, 0) != 1 /*If @SkipValidateLogins hasn't been set to 1 by the caller*/
+			BEGIN
+				IF EXISTS
+				(
+					SELECT	1/0
+					FROM	fn_my_permissions(N'sp_validatelogins', N'OBJECT') AS fmp
+					WHERE	fmp.permission_name = N'EXECUTE'
+				)
+				BEGIN
+					BEGIN TRY 
+						EXEC sp_validatelogins;
+                        
+						SET @SkipValidateLogins = 0 /*We can execute sp_validatelogins*/
+					END TRY
+					BEGIN CATCH
+						SET @SkipValidateLogins = 1 /*We have execute rights but sp_validatelogins throws an error so skip it*/
+					END CATCH
+				END;
+			END;
+			ELSE
+			BEGIN
+				SET @SkipValidateLogins = 1;
+			END; /*Need execute on sp_validatelogins*/
 
 			IF ISNULL(@SkipModel, 0) != 1 /*If @SkipModel hasn't been set to 1 by the caller*/
 			BEGIN