php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Request #73040 Remove usages of getdtablesize from ext/standard/php_fopen_wrapper.c
Submitted: 2016-09-07 14:48 UTC Modified: 2016-09-07 17:06 UTC
Votes:2
Avg. Score:5.0 ± 0.0
Reproduced:1 of 1 (100.0%)
Same Version:0 (0.0%)
Same OS:0 (0.0%)
From: subins2000 at gmail dot com Assigned:
Status: Verified Package: Compile Failure
PHP Version: 7.1.0RC1 OS: Ubuntu 16.04 Xenial Xerus
Private report: No CVE-ID: None
View Add Comment Developer Edit
Anyone can comment on a bug. Have a simpler test case? Does it work for you on a different platform? Let us know!
Just going to say 'Me too!'? Don't clutter the database with that please — but make sure to vote on the bug!
Your email address:
MUST BE VALID
Solve the problem:
48 - 8 = ?
Subscribe to this entry?

 
 [2016-09-07 14:48 UTC] subins2000 at gmail dot com
Description:
------------
The getdtablesize function is removed since 2004, here is the link: https://code.google.com/p/android/issues/detail?id=74387

It should be replaced with OPEN_MAX


Patches

replace-getdtablesize (last revision 2016-09-07 14:48 UTC by subins2000 at gmail dot com)

Add a Patch

Pull Requests

Add a Pull Request

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2016-09-07 16:15 UTC] cmb@php.net
-Status: Open +Status: Verified
 [2016-09-07 16:15 UTC] cmb@php.net
Good catch, thanks for reporting! This issue might even be
regarded as being a bug.

However, according to the Open Group
specification of getdtablesize()[1], getrlimit() should be called
instead of using OPEN_MAX.

[1] <http://pubs.opengroup.org/onlinepubs/7908799/xsh/getdtablesize.html>
 [2016-09-07 17:06 UTC] subins2000 at gmail dot com
Okay.

I caught this error while cross compiling for Android. Android has already removed getdtablesize()
 
PHP Copyright © 2001-2021 The PHP Group
All rights reserved.
Last updated: Sun Jan 17 20:01:23 2021 UTC