From 837541e93ab0eb2133b783c96f2b01e9f93a6079 Mon Sep 17 00:00:00 2001
From: Damien Regad <dregad@mantisbt.org>
Date: Wed, 18 Jun 2014 02:43:37 +0200
Subject: [PATCH] Fix PHP Notice Undefined offset

---
 browser_check.php | 36 ++++++++++++++++++------------------
 1 file changed, 18 insertions(+), 18 deletions(-)

diff --git a/browser_check.php b/browser_check.php
index b76e0ab..66d075b 100644
--- a/browser_check.php
+++ b/browser_check.php
@@ -8,7 +8,6 @@
  */
 function getBrowser() {
 	$u_agent = strtolower( $_SERVER['HTTP_USER_AGENT'] );
-	$version = '';
 
 	# First get the platform
 	if( preg_match( '/linux/', $u_agent ) ) {
@@ -47,26 +46,27 @@ function getBrowser() {
 	$known = array( 'version', $ub, 'other' );
 	$pattern = '#(?<browser>' . join( '|', $known ) . ')[/ ]+(?<version>[0-9.|a-zA-Z.]*)#';
 	if( !preg_match_all( $pattern, $u_agent, $matches ) ) {
-		# we have no matching number just continue
-	}
-
-	# see how many we have
-	$i = count( $matches['browser'] );
-	if( $i != 1 ) {
-		# we will have two since we are not using 'other' argument yet
-		# see if version is before or after the name
-		if( strrpos( $u_agent, 'version' ) < strrpos( $u_agent, $ub ) ) {
-			$version = $matches['version'][0];
+		# we have no matching number
+		$version = '?';
+	} else {
+		# see how many we have
+		$i = count( $matches['browser'] );
+		if( $i >= 1 ) {
+			# we will have two since we are not using 'other' argument yet
+			# see if version is before or after the name
+			if( strrpos( $u_agent, 'version' ) < strrpos( $u_agent, $ub ) ) {
+				$version = $matches['version'][0];
+			} else {
+				$version = $matches['version'][1];
+			}
 		} else {
-			$version = $matches['version'][1];
+			$version = $matches['version'][0];
 		}
-	} else {
-		$version = $matches['version'][0];
-	}
 
-	# check if we have a number
-	if( $version == '' ) {
-		$version = '?';
+		# check if we have a number
+		if( $version == '' ) {
+			$version = '?';
+		}
 	}
 
 	return array(