Fix #65550: get_browser() incorrectly parsers entries with "+" sign

+ signs in the browscap patterns are meant to be literal characters, so we
have to escape them for the regex matching.

Author: cmb69

NEWS

@@ -44,6 +44,8 @@ PHP                                                                        NEWS
 - Standard:
   . Fixed bug #72823 (strtr out-of-bound access). (cmb)
   . Fixed bug #72278 (getimagesize returning FALSE on valid jpg). (cmb)
+  . Fixed bug #65550 (get_browser() incorrectly parses entries with "+" sign).
+    (cmb)
 
 - XML:
   . Fixed bug #72085 (SEGV on unknown address zif_xml_parse). (cmb)

ext/standard/browscap.c

@@ -123,6 +123,10 @@ static void convert_browscap_pattern(zval *pattern, int persistent) /* {{{ */
 				t[j++] = '\\';
 				t[j] = '~';
 				break;
+			case '+':
+				t[j++] = '\\';
+				t[j] = '+';
+				break;
 			default:
 				t[j] = Z_STRVAL_P(pattern)[i];
 				break;

ext/standard/tests/misc/bug65550.phpt

@@ -0,0 +1,19 @@
+--TEST--
+Bug #65550 (get_browser() incorrectly parses entries with "+" sign)
+--INI--
+browscap={PWD}/browscap.ini
+--SKIPIF--
+<?php
+if (!is_readable(ini_get('browscap'))) die('skip browscap.ini file ' . ini_get('browscap') . ' not readable');
+?>
+--FILE--
+<?php
+$user_agent = 'Mozilla/5.0 (Macintosh; U; PPC Mac OS X; en-US) AppleWebKit/522+ (KHTML, like Gecko, Safari/522) OmniWeb/v613';
+$caps = get_browser($user_agent, true);
+var_dump($caps['browser'], $caps['version']);
+?>
+==DONE==
+--EXPECT--
+string(7) "OmniWeb"
+string(3) "5.6"
+==DONE==