iliaa Thu Aug 26 19:36:05 2004 EDT
Modified files:
/php-src/ext/zlib zlib_fopen_wrapper.c
Log:
Fixed a nasty resource/memory/disk-space leak when opening zlib files
via wrapper stacking ala compress.zlib://http://...
http://cvs.php.net/diff.php/php-src/ext/zlib/zlib_fopen_wrapper.c?r1=1.44&r2=1.45&ty=u
Index: php-src/ext/zlib/zlib_fopen_wrapper.c
diff -u php-src/ext/zlib/zlib_fopen_wrapper.c:1.44
php-src/ext/zlib/zlib_fopen_wrapper.c:1.45
--- php-src/ext/zlib/zlib_fopen_wrapper.c:1.44 Thu Jan 8 03:17:52 2004
+++ php-src/ext/zlib/zlib_fopen_wrapper.c Thu Aug 26 19:36:05 2004
@@ -17,7 +17,7 @@
+----------------------------------------------------------------------+
*/
-/* $Id: zlib_fopen_wrapper.c,v 1.44 2004/01/08 08:17:52 andi Exp $ */
+/* $Id: zlib_fopen_wrapper.c,v 1.45 2004/08/26 23:36:05 iliaa Exp $ */
#define _GNU_SOURCE
@@ -27,6 +27,7 @@
struct php_gz_stream_data_t {
gzFile gz_file;
+ php_stream *stream;
};
static size_t php_gziop_read(php_stream *stream, char *buf, size_t count TSRMLS_DC)
@@ -74,6 +75,10 @@
ret = gzclose(self->gz_file);
self->gz_file = NULL;
}
+ if (self->stream) {
+ php_stream_close(self->stream);
+ self->stream = NULL;
+ }
}
efree(self);
@@ -100,7 +105,7 @@
php_stream *php_stream_gzopen(php_stream_wrapper *wrapper, char *path, char *mode,
int options,
char **opened_path,
php_stream_context *context STREAMS_DC TSRMLS_DC)
{
- struct php_gz_stream_data_t *self;
+ struct php_gz_stream_data_t *self = {0};
php_stream *stream = NULL, *innerstream = NULL;
/* sanity check the stream: it can be either read-only or write-only */
@@ -124,8 +129,9 @@
if (innerstream) {
int fd;
- if (SUCCESS == php_stream_cast(innerstream, PHP_STREAM_AS_FD |
PHP_STREAM_CAST_RELEASE, (void **) &fd, REPORT_ERRORS)) {
+ if (SUCCESS == php_stream_cast(innerstream, PHP_STREAM_AS_FD, (void
**) &fd, REPORT_ERRORS)) {
self->gz_file = gzdopen(fd, mode);
+ self->stream = innerstream;
if (self->gz_file) {
stream = php_stream_alloc_rel(&php_stream_gzio_ops,
self, 0, mode);
if (stream) {
--
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php
