* in the voicemail main loop vms->curmsg
initial invalid pointer is set to -1 to cover
scenarios where user presses '4' to listen a previous
message, otherwise the pointer produces playback of an
invalid message with the invalid counter
Change-Id: If3c69b1e1783b25e4e1fd10ecf96e0923a7178ce
1) pcre_exec takes the number of elements in the ovec array, not its
size in bytes
2) fix off-by-one error in matching and replacing substrings
This bug was triggered by a match string of "^00|\\+([1-9][0-9]+)$" and
a replacement string of "$1" against a subject string "0000".
pcre_exec would return 1, with the single substring capture in ovec
being unset. The code would try to extract the $1 match but would run
into uninitialised values.
Change-Id: I3898fa8b807994b0b65fefb3f61cc9756180c490
* pcre_malloc() is changed to ast_malloc() as ast_free()
is used to free up the memory block
* struct vm_state when used now sets pointers to NULL
Change-Id: Ibd65790f7b34e19a4145c86f63b89e2568ea95a3
* port app_voicemail and app_playback
to support Asterisk 16 (Debian/buster)
* rename .diff files into .patch for consistency
Change-Id: Ic5260e68f5a77d1640480e338dad44de4ae3b789
* envelope playback was disabled in the options,
now reenabled back.
* fix an issue when DTMF delay causing entering a
suboption "advanced_options" straight away,
e.g.: pressing '3' from the message playback
entered "advanced_options" + immediately started
with the menu item '3'. Now the options for
"advanced options" are played correctly.
Change-Id: I8aa6f5fc9a022cf86218c8be3a6b3645a3f352e3
* store_file: no longer uses the implicit connection
object check as it is checked by res_odbc
* store_file: add diag info (error code + error message)
for failed sql statements
Change-Id: I145b056ea8787b47864a661f40ca10a3717fe060
* store_file uses direct ODBC commands instead of
ast_odbc_prepare_and_execute, and therefore,
no checks if the ODBC object is not connected anymore.
the connection object is now checked if it is actually
connected before issuing the store message SQL.
Change-Id: I0d6b607728c22948b88013225c07af516905d3cc
New patch debian/patches/sipwise_fix_chan_usage.patch on top of
sipwise_vm_uuid_mailbox.patch, to address build failure on
Debian/buster.
chan is defined only for IMAP_STORAGE inside
AST_TEST_DEFINE(test_voicemail_msgcount), so accordingly invoke STORE()
also iff IMAP_STORAGE is defined.
Change-Id: Ie7e255f1c977934cb6351d0a1010826f23965778