]> granicus.if.org Git - php/commitdiff
- Fixed bug #48240 (DBA Segmentation fault dba_nextkey)
authorFelipe Pena <felipe@php.net>
Wed, 13 May 2009 02:14:18 +0000 (02:14 +0000)
committerFelipe Pena <felipe@php.net>
Wed, 13 May 2009 02:14:18 +0000 (02:14 +0000)
ext/dba/dba_db4.c
ext/dba/tests/bug48240.phpt [new file with mode: 0644]

index 5cbdd649061b84a1e2a3076a67380fb81d7dc660..df20ac7d1e5e8524dbf2fc2d717911360f63115d 100644 (file)
@@ -219,7 +219,7 @@ DBA_NEXTKEY_FUNC(db4)
                gkey.flags |= DB_DBT_MALLOC;
                gval.flags |= DB_DBT_MALLOC;
        }
-       if (dba->cursor->c_get(dba->cursor, &gkey, &gval, DB_NEXT) == 0) {
+       if (dba->cursor && dba->cursor->c_get(dba->cursor, &gkey, &gval, DB_NEXT) == 0) {
                if (gkey.data) {
                        nkey = estrndup(gkey.data, gkey.size);
                        if (newlen) *newlen = gkey.size;
diff --git a/ext/dba/tests/bug48240.phpt b/ext/dba/tests/bug48240.phpt
new file mode 100644 (file)
index 0000000..72dc439
--- /dev/null
@@ -0,0 +1,25 @@
+--TEST--
+Bug #48240 (DBA Segmentation fault dba_nextkey)
+--SKIPIF--
+<?php 
+       $handler = 'db4';
+       require_once('skipif.inc');
+?>
+--FILE--
+<?php
+
+$handler = 'db4';
+require_once('test.inc');
+
+$db = dba_open($db_filename, 'c', 'db4');
+
+var_dump(dba_nextkey($db));
+
+dba_close($db);
+unlink($db_filename);
+
+?>
+===DONE===
+--EXPECT--
+bool(false)
+===DONE===