|
|
php.net | support | documentation | report a bug | advanced search | search howto | statistics | random bug | login |
[2003-06-03 06:35 UTC] per at nobolt dot com
In the bz2_aware_stream_open(), there is code like this:
BCOMPILER_DEBUG(("no bz2 support - opening it..\n"));
stream = php_stream_open_wrapper(file_name, "rb", ENFORCE_SAFE_MODE | REPORT_ERRORS, NULL);
/* Sanity check to see if it is a bzip2 encoded stream */
php_stream_read(stream, magic, sizeof(magic));
If the call to php_stream_open_wrapper() fails (which it will if the file does not exist), stream will be NULL and this will cause a segmentation fault in php_stream_read().
This can easily be fixed by just checking the return value; if it is NULL, return from the function immediately.
PatchesPull RequestsHistoryAllCommentsChangesGit/SVN commits
|
|||||||||||||||||||||||||||
Copyright © 2001-2025 The PHP GroupAll rights reserved. |
Last updated: Mon Oct 27 19:00:02 2025 UTC |
This bug is very serious since it causes segmentation faults on every missing file in include(), require() etc. (since bcompiler hooks into the Zend core). This patch will fix it: Index: bcompiler.c =================================================================== RCS file: /repository/pear/PECL/bcompiler/bcompiler.c,v retrieving revision 1.39 diff -u -r1.39 bcompiler.c --- bcompiler.c 8 Apr 2003 07:59:44 -0000 1.39 +++ bcompiler.c 3 Jun 2003 12:29:51 -0000 @@ -215,6 +215,10 @@ BCOMPILER_DEBUG(("no bz2 support - opening it..\n")); stream = php_stream_open_wrapper(file_name, "rb", ENFORCE_SAFE_MODE | REPORT_ERRORS, NULL); + if (!stream) { + return stream; + } + /* Sanity check to see if it is a bzip2 encoded stream */ php_stream_read(stream, magic, sizeof(magic)); if (memcmp(magic, "BZ", 2) == 0) { The only problem with this patch is that you get the warning about the missing file twice, but IMO that's much better than a segmentation fault.