From 73dcea59bd17c694e636a886e151863c02311822 Mon Sep 17 00:00:00 2001 From: Matthew Jordan Date: Thu, 19 Mar 2015 19:19:51 +0000 Subject: funcs/func_env: Fix regression caused in FILE read operation When r432935 was merged, it did correctly fix a situation where a FILE read operation on the middle of a file buffer would not read the requested length in the parameters passed to the FILE function. Unfortunately, it would also allow the FILE function to append more bytes than what was available in the buffer if the length exceeded the end of the buffer length. This patch takes the minimum of the remaining bytes in the buffer along with the calculated length to append provided by the original patch, and uses that as the length to append in the return result. This patch also updates the unit tests with the scenarios that were originally pointed out in ASTERISK-21765 that the original implementation treated incorrectly. ASTERISK-21765 ........ Merged revisions 433173 from http://svn.asterisk.org/svn/asterisk/branches/11 git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/13@433174 65c4cc65-6c06-0410-ace0-fbb531ad65f3 --- tests/test_func_file.c | 6 ++++++ 1 file changed, 6 insertions(+) (limited to 'tests') diff --git a/tests/test_func_file.c b/tests/test_func_file.c index 4ebc87caf..834907578 100644 --- a/tests/test_func_file.c +++ b/tests/test_func_file.c @@ -63,6 +63,12 @@ static struct { /* No length */ { "123456789", "-5", "56789" }, { "123456789", "4", "56789" }, + /* Passed file length */ + { "123456789", "8,10", "9" }, + { "123456789", "10,1", "" }, + /* Middle of file */ + { "123456789", "2,5", "34567" }, + { "123456789", "-7,5", "34567" }, /* Line mode, 4 ways of specifying the first character */ { "123\n456\n789\n", "0,1,l", "123\n" }, { "123\n456\n789\n", "-3,1,l", "123\n" }, -- cgit v1.2.3