#45788 [NEW]: fgetcsv retuns NULL on warning instead of false

From:             steven at realestatewebmasters dot com
Operating system: Linux
PHP version:      5.2.6
PHP Bug Type:     Filesystem function related
Bug description:  fgetcsv retuns NULL on warning instead of false

Description:
------------
fgetcsv returns null if passed false as the handle. This should cause and
error instead of a warning and cause the function to return false.

Reproduce code:
---------------
<?php

$handle = fopen("foo","r");

var_dump($data = fgetcsv($handle,0,",",'"'));

while (($data = fgetcsv($handle,0,",",'"')) !== false) {
	echo 'test';
}

?>

Expected result:
----------------
Warning: fopen(foo): failed to open stream: No such file or directory in
/home/sbarre85/test.php on line 3

Warning: fgetcsv() expects parameter 1 to be resource, boolean given in
/home/sbarre85/test.php on line 5
bool(false)

Error: fgetcsv() expects parameter 1 to be resource, boolean given in
/home/sbarre85/test.php on line 7

Actual result:
--------------
Warning: fopen(foo): failed to open stream: No such file or directory in
/home/sbarre85/test.php on line 3

Warning: fgetcsv() expects parameter 1 to be resource, boolean given in
/home/sbarre85/test.php on line 5
NULL

Warning: fgetcsv() expects parameter 1 to be resource, boolean given in
/home/sbarre85/test.php on line 7
test

.. looping infinitly

-- 
Edit bug report at http://bugs.php.net/?id=45788&edit=1
-- 
Try a CVS snapshot (PHP 5.2): http://bugs.php.net/fix.php?id=45788&r=trysnapshot52
Try a CVS snapshot (PHP 5.3): http://bugs.php.net/fix.php?id=45788&r=trysnapshot53
Try a CVS snapshot (PHP 6.0): http://bugs.php.net/fix.php?id=45788&r=trysnapshot60
Fixed in CVS:                 http://bugs.php.net/fix.php?id=45788&r=fixedcvs
Fixed in release:             http://bugs.php.net/fix.php?id=45788&r=alreadyfixed
Need backtrace:               http://bugs.php.net/fix.php?id=45788&r=needtrace
Need Reproduce Script:        http://bugs.php.net/fix.php?id=45788&r=needscript
Try newer version:            http://bugs.php.net/fix.php?id=45788&r=oldversion
Not developer issue:          http://bugs.php.net/fix.php?id=45788&r=support
Expected behavior:            http://bugs.php.net/fix.php?id=45788&r=notwrong
Not enough info:              http://bugs.php.net/fix.php?id=45788&r=notenoughinfo
Submitted twice:              http://bugs.php.net/fix.php?id=45788&r=submittedtwice
register_globals:             http://bugs.php.net/fix.php?id=45788&r=globals
PHP 4 support discontinued:   http://bugs.php.net/fix.php?id=45788&r=php4
Daylight Savings:             http://bugs.php.net/fix.php?id=45788&r=dst
IIS Stability:                http://bugs.php.net/fix.php?id=45788&r=isapi
Install GNU Sed:              http://bugs.php.net/fix.php?id=45788&r=gnused
Floating point limitations:   http://bugs.php.net/fix.php?id=45788&r=float
No Zend Extensions:           http://bugs.php.net/fix.php?id=45788&r=nozend
MySQL Configuration Error:    http://bugs.php.net/fix.php?id=45788&r=mysqlcfg