Edit report at http://bugs.php.net/bug.php?id=53709&edit=1
ID: 53709
Comment by: jthijssen at noxlogic dot nl
Reported by: jthijssen at noxlogic dot nl
Summary: [feat] md5_file and sha1_file should accept an
additional length
Status: Open
Type: Feature/Change Request
Package: Strings related
Operating System: NA
PHP Version: 5.3SVN-2011-01-10 (SVN)
Block user comment: N
Private report: N
New Comment:
I've added the patch in a base64 format because of a bug in the php
bugtracker (bug #53703).
Previous Comments:
------------------------------------------------------------------------
[2011-01-10 22:11:44] jthijssen at noxlogic dot nl
Description:
------------
Sometimes you want to have the md5() or sha1() from a file, but not the
whole file. This is particularly handy when you want to sync directories
and you don't want to md5 whole files but just the first 4096 or 8192
bytes to speed up the checking.
This patch implements an additional parameter to md5_file() and
sha1_file() where you can add the number of bytes that needs to be
read/md5()'d from the source file.
Test script:
---------------
Returns the md5 from the first 8192 bytes from the iso-file:
md5_file('dvdimage.iso', false, 8192);
Returns the md5 from the complete file, even though length is way larger
than the file:
md5_file('dvdimage.iso', false, 81920000000);
Expected result:
----------------
The md5_file() and sha1_file() functions actually accepting the
additional parameter.
Actual result:
--------------
Currently, it does not accept a 3rd paramter. The patch adds the
additional parameter (which defaults to -1, which means to read the
complete file).
Additionally added and fixed tests in md5_file.phpt and sha1_file.phpt
------------------------------------------------------------------------
--
Edit this bug report at http://bugs.php.net/bug.php?id=53709&edit=1
