| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This patch adds missing newlines, trims multiple redundant final
newlines into a single one, and trims redundant leading newlines.
According to POSIX, a line is a sequence of zero or more non-' <newline>'
characters plus a terminating '<newline>' character. [1] Files should
normally have at least one final newline character.
C89 [2] and later standards [3] mention a final newline:
"A source file that is not empty shall end in a new-line character,
which shall not be immediately preceded by a backslash character."
Although it is not mandatory for all files to have a final newline
fixed, a more consistent and homogeneous approach brings less of commit
differences issues and a better development experience in certain text
editors and IDEs.
[1] http://pubs.opengroup.org/onlinepubs/9699919799/basedefs/V1_chap03.html#tag_03_206
[2] https://port70.net/~nsz/c/c89/c89-draft.html#2.1.1.2
[3] https://port70.net/~nsz/c/c99/n1256.html#5.1.1.2
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This patch simplifies line endings tracked in the Git repository and
syncs them to all include the LF style instead of the CRLF files.
Newline characters:
- LF (\n) (*nix and Mac)
- CRLF (\r\n) (Windows)
- CR (\r) (old Mac, obsolete)
To see which line endings are in the index and in the working copy the
following command can be used:
`git ls-files --eol`
Git additionally provides `.gitattributes` file to specify if some files
need to have specific line endings on all platforms (either CRLF or LF).
Changed files shouldn't cause issues on modern Windows platforms because
also Git can do output conversion is core.autocrlf=true is set on
Windows and use CRLF newlines in all files in the working tree.
Unless CRLF files are tracked specifically, Git by default tracks all
files in the index using LF newlines.
|
|\
| |
| |
| |
| | |
* PHP-7.1:
Remove unstable dns tests
|
| | |
|
| | |
|
|\ \
| |/
| |
| |
| | |
* PHP-7.1:
Fix bug #74764 and add a test case
|
| | |
|
| |
| |
| |
| |
| |
| |
| | |
from 7.3
It would probably be better to remove this test as it's too much
maintanance for a little gain.
|
|\ \
| |/ |
|
| |
| |
| |
| | |
Correctly report errors when opening FTP data connection.
|
|\ \
| |/
| |
| |
| | |
* PHP-7.1:
Simplify regression test
|
| |
| |
| |
| |
| |
| |
| | |
There's no need to actually try to trigger an out-of-memory condition
to proof the leak; instead we can simply rely on the Zend MM to report
the memory leaks in debug mode (at least on Linux). Therefore we
simplify the regression test, which also makes it run much faster.
|
|\ \
| |/
| |
| |
| | |
* PHP-7.1:
Increase memory_limit to prevent test failures
|
| | |
|
|\ \
| |/
| |
| |
| | |
* PHP-7.1:
Fix #76778: array_reduce leaks memory if callback throws exception
|
| |
| |
| |
| | |
We have to release the result variable in the error case, too.
|
|\ \
| |/
| |
| |
| | |
* PHP-7.1:
Fix #76643: Segmentation fault when using `output_add_rewrite_var`
|
| |
| |
| |
| |
| | |
We have to check whether _SERVER is actually an array before we're
going to use it as such.
|
|\ \
| |/
| |
| |
| | |
* PHP-7.1:
Fixed bug #68553 (array_column: null values in $index_key become incrementing keys in result)
|
| |
| |
| |
| | |
incrementing keys in result)
|
|\ \
| |/
| |
| |
| | |
* PHP-7.1:
Fix #73817: Incorrect entries in get_html_translation_table
|
| |
| |
| |
| |
| |
| |
| |
| | |
Due to incorrect string termination and length handling, several HTML
entities missed the trailing semicolon.
We also fix the obviously wrong expectations in two already existing
tests.
|
|\ \
| |/
| |
| |
| | |
* PHP-7.1:
Fixed RecursiveDirectoryIterator with long path or with edge case length
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
The search path needs to be appended with the wild card. Till now, an
edge case existed, so then if a path is 259 bytes long, which is smaller
_MAX_PATH, the suffix would cause the final search path to become longer
than _MAX_PATH. It is an edge case, when the starting path happens to
have a specific length. If the starting path was longer than _MAX_PATH
or the addition of "\\*" would not exceed _MAX_PATH, the function was
correct. Except for rewind, which was broken in the case of the long
path.
|
|\ \
| |/ |
|
| |
| |
| |
| |
| | |
Was segfaulting if no allowed tags are specified and performing an
out of bounds read if they were.
|
|\ \
| |/ |
|
| |
| |
| |
| | |
And move it to a more appropriate location.
|
|\ \
| |/ |
|
| |
| |
| |
| |
| |
| |
| |
| | |
Validate that "C" serialization payload is followed by "}" prior to
calling the unserialize() handler. This mitigates issues caused by
unserialize() not correctly handling strings that are not NUL
terminated. Making sure that there is a "}" at the end avoids the
problem.
|
|\ \
| |/
| |
| |
| | |
* PHP-7.1:
Fix #71848: getimagesize with $imageinfo returns false
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Some JFIF images contain empty APP segments, i.e. those which consist
only of the marker bytes and the length, but without actual content.
It appears to be doubtful to have empty APP segments, but we should
apply the robustness principle, and accept these, instead of simply
failing in this case.
We choose to add empty APP segments to $imageinfo with an empty string
as value, instead of NULL, or even to omit these segments altogether.
This patch also fixes the potential issue that php_stream_read() might
not read the supposed number of bytes, which could result in garbage to
be added to the read value.
|
|\ \
| |/
| |
| |
| | |
* PHP-7.1:
Fixed bug #76505 (array_merge_recursive() is duplicating sub-array keys)
|
| | |
|
| | |
|
|\ \
| |/
| |
| |
| | |
* PHP-7.1:
Fix bug #76390 - do not allow invalid strings in range()
|
| | |
|
|\ \
| |/
| |
| |
| | |
* PHP-7.1:
Fixed bug #76335 "link(): Bad file descriptor" with non-ASCII path
|
| | |
|
| | |
|
|\ \
| |/
| |
| |
| | |
* PHP-7.1:
Sync test with newer output
|
| |
| |
| |
| | |
Czech Republic vs. newer Czechia
|
|\ \
| |/
| |
| |
| | |
* PHP-7.1:
Fixed bug #76068 parse_ini_string fails to parse "[foo]\nbar=1|>baz" with segfault
|
| |
| |
| |
| | |
segfault
|
|\ \
| |/ |
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
This reverts commit 94e9d0a2ae76bad712495d820d3962e401085fef.
This code needs to be mindful about modifications to the array
happening during callback execution. It was written in a way that
only accessed the reference, which is guaranteed not to move. The
changed implementation instead accesses the array slot, leading to
use-after-free.
Run ext/standard/tests/array/bug61967.phpt under valgrind to see
the issue.
|
|\ \
| |/
| |
| |
| | |
* PHP-7.1:
Fix bug #75981: prevent reading beyond buffer start
|
| |\
| | |
| | |
| | |
| | | |
* PHP-7.0:
Fix bug #75981: prevent reading beyond buffer start
|