Skip to content

Commit d14b208

Browse files
committed
Merge branch 'PHP-8.3'
* PHP-8.3: Fix GH-13984: Buffer size is now checked before memcmp (#13991)
2 parents e23440e + dd5cdfd commit d14b208

File tree

2 files changed

+19
-1
lines changed

2 files changed

+19
-1
lines changed

ext/pdo_sqlite/sqlite_driver.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -750,7 +750,7 @@ static char *make_filename_safe(const char *filename)
750750
}
751751
return estrdup(filename);
752752
}
753-
if (*filename && memcmp(filename, ":memory:", sizeof(":memory:"))) {
753+
if (*filename && strcmp(filename, ":memory:")) {
754754
char *fullpath = expand_filepath(filename, NULL);
755755

756756
if (!fullpath) {

ext/pdo_sqlite/tests/gh13991.phpt

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,18 @@
1+
--TEST--
2+
Fix GH-13984: Buffer size is now checked before memcmp
3+
--EXTENSIONS--
4+
pdo_sqlite
5+
--SKIPIF--
6+
<?php if (file_exists(getcwd() . '/13991db')) die('skip File "13991db" already exists.'); ?>
7+
--FILE--
8+
<?php
9+
$dbfile = '13991db';
10+
$db = new PDO('sqlite:' . $dbfile, null, null, [PDO::ATTR_PERSISTENT => true]);
11+
echo 'done!';
12+
?>
13+
--CLEAN--
14+
<?php
15+
@unlink(getcwd() . '/13991db');
16+
?>
17+
--EXPECT--
18+
done!

0 commit comments

Comments
 (0)