Fix Issue #59 segfault on null passed as handle to uv_stdio_new

Author: steverhoades

php_uv.c

@@ -4908,11 +4908,10 @@ PHP_FUNCTION(uv_stdio_new)
 		return;
 	}
 
-	stdio = (php_uv_stdio_t*)emalloc(sizeof(php_uv_stdio_t));
-	stdio->flags = flags;
-	stdio->stream = NULL;
-
-	if (Z_TYPE_P(handle) == IS_LONG) {
+	if(Z_TYPE_P(handle) == IS_NULL) {
+		php_error_docref(NULL TSRMLS_CC, E_WARNING, "passed unexpected resource");
+		RETURN_FALSE;
+    } else if (Z_TYPE_P(handle) == IS_LONG) {
 		fd = Z_LVAL_P(handle);
 	} else if (Z_TYPE_P(handle) == IS_RESOURCE) {
 		if (ZEND_FETCH_RESOURCE_NO_RETURN(stream, php_stream*, &handle, -1, NULL, php_file_le_stream())) {
@@ -4929,7 +4928,10 @@ PHP_FUNCTION(uv_stdio_new)
 		}
 	}
 
-	
+    stdio = (php_uv_stdio_t*)emalloc(sizeof(php_uv_stdio_t));
+	stdio->flags = flags;
+	stdio->stream = NULL;
+
 	stdio->fd = fd;
 
 	if (Z_TYPE_P(handle) == IS_RESOURCE) {

tests/800-uv_spawn-issue59.phpt

@@ -0,0 +1,9 @@
+--TEST--
+Test uv_spawn doesn't cause segfault #56
+--FILE--
+<?php
+
+$ioRead = uv_stdio_new(null, Uv::CREATE_PIPE | Uv::INHERIT_STREAM);
+
+--EXPECTF--
+Warning: uv_stdio_new(): passed unexpected resource in %s on line %d