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