php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Doc Bug #77781 fopen() b flag recommendation no longer necessary?
Submitted: 2019-03-22 12:04 UTC Modified: -
From: martin at auswoeger dot com Assigned:
Status: Open Package: Filesystem function related
PHP Version: Irrelevant OS:
Private report: No CVE-ID: None
Welcome back! If you're the original bug submitter, here's where you can edit the bug or add additional notes.
If this is not your bug, you can add a comment by following this link.
If this is your bug, but you forgot your password, you can retrieve your password here.
Password:
Status:
Package:
Bug Type:
Summary:
From: martin at auswoeger dot com
New email:
PHP Version: OS:

 

 [2019-03-22 12:04 UTC] martin at auswoeger dot com
Description:
------------
---
From manual page: https://php.net/function.fopen
---

The documentation for fopen() includes the following note:

For portability, it is strongly recommended that you always use the 'b' flag when opening files with fopen().

However it seems that since PHP 4.3.2 the 'b' flag is already the default and it should no longer be necessary.

Looking at the source code it also seems that the 'b' flag is not necessary anymore:  
- https://github.com/php/php-src/blob/php-7.3.2/main/streams/plain_wrapper.c#L110-L114  
- https://github.com/php/php-src/commit/fa342d2404a9c809d0a182d80798da2abd3a98f7

I think this recommendation note should be removed.


Patches

Add a Patch

Pull Requests

Add a Pull Request

 
PHP Copyright © 2001-2019 The PHP Group
All rights reserved.
Last updated: Sat Sep 21 17:01:27 2019 UTC