jani Mon Nov 5 16:12:55 2007 UTC Modified files: /php-src/ext/standard filestat.c Log: ws + cs + sync some fixes from PHP_5_3 branch http://cvs.php.net/viewvc.cgi/php-src/ext/standard/filestat.c?r1=1.165&r2=1.166&diff_format=u Index: php-src/ext/standard/filestat.c diff -u php-src/ext/standard/filestat.c:1.165 php-src/ext/standard/filestat.c:1.166 --- php-src/ext/standard/filestat.c:1.165 Wed Oct 31 13:22:18 2007 +++ php-src/ext/standard/filestat.c Mon Nov 5 16:12:55 2007 @@ -16,7 +16,7 @@ +----------------------------------------------------------------------+ */ -/* $Id: filestat.c,v 1.165 2007/10/31 13:22:18 jani Exp $ */ +/* $Id: filestat.c,v 1.166 2007/11/05 16:12:55 jani Exp $ */ #include "php.h" #include "fopen_wrappers.h" @@ -227,7 +227,7 @@ #endif /* }}} */ /* }}} */ - + /* {{{ proto float disk_total_space(string path) U Get total disk space for filesystem that path is on */ PHP_FUNCTION(disk_total_space) @@ -434,13 +434,17 @@ if (Z_TYPE_P(group) == IS_LONG) { gid = (gid_t)Z_LVAL_P(group); } else { -#if defined(ZTS) && defined(HAVE_GETGRNAM_R) && defined(_SC_GETGR_R_SIZE_MAX) +#if defined(ZTS) && defined(HAVE_GETGRNAM_R) && defined(_SC_GETGR_R_SIZE_MAX) struct group gr; struct group *retgrptr; - int grbuflen = sysconf(_SC_GETGR_R_SIZE_MAX); - char *grbuf = emalloc(grbuflen); + long grbuflen = sysconf(_SC_GETGR_R_SIZE_MAX); + char *grbuf; + + if (grbuflen < 1) { + RETURN_FALSE; + } - convert_to_string(group); + grbuf = emalloc(grbuflen); if (getgrnam_r(Z_STRVAL_P(group), &gr, grbuf, grbuflen, &retgrptr) != 0 || retgrptr == NULL) { php_error_docref(NULL TSRMLS_CC, E_WARNING, "Unable to find gid for %s", Z_STRVAL_P(group)); efree(grbuf); @@ -449,9 +453,9 @@ efree(grbuf); gid = gr.gr_gid; #else - struct group *gr; - convert_to_string(group); - if (!(gr = getgrnam(Z_STRVAL_P(group)))) { + struct group *gr = getgrnam(Z_STRVAL_P(group)); + + if (!gr) { php_error_docref(NULL TSRMLS_CC, E_WARNING, "Unable to find gid for %s", Z_STRVAL_P(group)); RETURN_FALSE; } @@ -539,13 +543,17 @@ if (Z_TYPE_P(user) == IS_LONG) { uid = (uid_t)Z_LVAL_P(user); } else { -#if defined(_SC_GETPW_R_SIZE_MAX) && defined(HAVE_GETPWNAM_R) +#if defined(ZTS) && defined(_SC_GETPW_R_SIZE_MAX) && defined(HAVE_GETPWNAM_R) struct passwd pw; struct passwd *retpwptr = NULL; - int pwbuflen = sysconf(_SC_GETPW_R_SIZE_MAX); - char *pwbuf = emalloc(pwbuflen); + long pwbuflen = sysconf(_SC_GETPW_R_SIZE_MAX); + char *pwbuf; - convert_to_string(user); + if (pwbuflen < 1) { + RETURN_FALSE; + } + + pwbuf = emalloc(pwbuflen); if (getpwnam_r(Z_STRVAL_P(user), &pw, pwbuf, pwbuflen, &retpwptr) != 0 || retpwptr == NULL) { php_error_docref(NULL TSRMLS_CC, E_WARNING, "Unable to find uid for %s", Z_STRVAL_P(user)); efree(pwbuf); @@ -554,10 +562,9 @@ efree(pwbuf); uid = pw.pw_uid; #else - struct passwd *pw; + struct passwd *pw = getpwnam(Z_STRVAL_P(user)); - convert_to_string(user); - if (!(pw = getpwnam(Z_STRVAL_P(user)))) { + if (!pw) { php_error_docref(NULL TSRMLS_CC, E_WARNING, "Unable to find uid for %s", Z_STRVAL_P(user)); RETURN_FALSE; } @@ -804,12 +811,14 @@ PHPAPI void php_stat(const char *filename, php_stat_len filename_length, int type, zval *return_value TSRMLS_DC) { zval *stat_dev, *stat_ino, *stat_mode, *stat_nlink, *stat_uid, *stat_gid, *stat_rdev, - *stat_size, *stat_atime, *stat_mtime, *stat_ctime, *stat_blksize, *stat_blocks; + *stat_size, *stat_atime, *stat_mtime, *stat_ctime, *stat_blksize, *stat_blocks; struct stat *stat_sb; php_stream_statbuf ssb; int flags = 0, rmask=S_IROTH, wmask=S_IWOTH, xmask=S_IXOTH; /* access rights defaults to other */ - char *stat_sb_names[13]={"dev", "ino", "mode", "nlink", "uid", "gid", "rdev", - "size", "atime", "mtime", "ctime", "blksize", "blocks"}; + char *stat_sb_names[13] = { + "dev", "ino", "mode", "nlink", "uid", "gid", "rdev", + "size", "atime", "mtime", "ctime", "blksize", "blocks" + }; char *local; php_stream_wrapper *wrapper; @@ -903,7 +912,7 @@ #ifndef NETWARE if (IS_ABLE_CHECK(type) && getuid() == 0) { - /* root has special perms on plain_wrapper + /* root has special perms on plain_wrapper But we don't know about root under Netware */ if (wrapper == &php_plain_files_wrapper) { if (type == FS_IS_X) { @@ -986,9 +995,9 @@ MAKE_LONG_ZVAL_INCREF(stat_uid, stat_sb->st_uid); MAKE_LONG_ZVAL_INCREF(stat_gid, stat_sb->st_gid); #ifdef HAVE_ST_RDEV - MAKE_LONG_ZVAL_INCREF(stat_rdev, stat_sb->st_rdev); + MAKE_LONG_ZVAL_INCREF(stat_rdev, stat_sb->st_rdev); #else - MAKE_LONG_ZVAL_INCREF(stat_rdev, -1); + MAKE_LONG_ZVAL_INCREF(stat_rdev, -1); #endif MAKE_LONG_ZVAL_INCREF(stat_size, stat_sb->st_size); #ifdef NETWARE @@ -1001,7 +1010,7 @@ MAKE_LONG_ZVAL_INCREF(stat_ctime, stat_sb->st_ctime); #endif #ifdef HAVE_ST_BLKSIZE - MAKE_LONG_ZVAL_INCREF(stat_blksize, stat_sb->st_blksize); + MAKE_LONG_ZVAL_INCREF(stat_blksize, stat_sb->st_blksize); #else MAKE_LONG_ZVAL_INCREF(stat_blksize,-1); #endif @@ -1017,7 +1026,7 @@ zend_hash_next_index_insert(HASH_OF(return_value), (void *)&stat_nlink, sizeof(zval *), NULL); zend_hash_next_index_insert(HASH_OF(return_value), (void *)&stat_uid, sizeof(zval *), NULL); zend_hash_next_index_insert(HASH_OF(return_value), (void *)&stat_gid, sizeof(zval *), NULL); - + zend_hash_next_index_insert(HASH_OF(return_value), (void *)&stat_rdev, sizeof(zval *), NULL); zend_hash_next_index_insert(HASH_OF(return_value), (void *)&stat_size, sizeof(zval *), NULL); zend_hash_next_index_insert(HASH_OF(return_value), (void *)&stat_atime, sizeof(zval *), NULL);
-- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php