php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Bug #57659 Error in config.w32
Submitted: 2007-05-09 09:32 UTC Modified: 2007-06-02 14:59 UTC
From: auroraeosrose at gmail dot com Assigned:
Status: Closed Package: yaz (PECL)
PHP Version: 5_2 CVS-2007-05-09 OS: WinXPSP2
Private report: No CVE-ID: None
View Add Comment Developer Edit
Anyone can comment on a bug. Have a simpler test case? Does it work for you on a different platform? Let us know!
Just going to say 'Me too!'? Don't clutter the database with that please !
Your email address:
MUST BE VALID
Solve the problem:
48 - 16 = ?
Subscribe to this entry?

 
 [2007-05-09 09:32 UTC] auroraeosrose at gmail dot com
Description:
------------
you need to check that what file_get_contents returns is a string before attempting a match on it or jscript will bail - patch attached

Reproduce code:
---------------
Index: config.w32
===================================================================
RCS file: /repository/pecl/yaz/config.w32,v
retrieving revision 1.3
diff -u -r1.3 config.w32
--- config.w32	21 Apr 2004 12:38:28 -0000	1.3
+++ config.w32	9 May 2007 13:24:51 -0000
@@ -15,7 +15,7 @@
 	}
 
 	c = file_get_contents(yaz_h + "\\yaz\\yaz-version.h");
-	if (c.match(/YAZ_VERSIONL\s+(0x[a-zA-Z0-9]+)/)) {
+	if (typeof(c) == "string" && c.match(/YAZ_VERSIONL\s+(0x[a-zA-Z0-9]+)/)) {
 		v = RegExp.$1;
 		if (parseInt(v) >= 0x2000D) {
 			return true;



Patches

Add a Patch

Pull Requests

Add a Pull Request

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2007-06-02 14:59 UTC] adam at indexdata dot dk
Patch applied to CVS.
 
PHP Copyright © 2001-2024 The PHP Group
All rights reserved.
Last updated: Fri Apr 26 08:01:30 2024 UTC