app_sms: Fix uninitialized values; hangup channel when REL is sent successfully
authorMatthew Jordan <mjordan@digium.com>
Sat, 19 Apr 2014 01:01:18 +0000 (01:01 +0000)
committerMatthew Jordan <mjordan@digium.com>
Sat, 19 Apr 2014 01:01:18 +0000 (01:01 +0000)
commit5547fb99ccdaaf5fe94738d67355d83049990813
treeb114f71e264a37e3562dbae948081d832dee68ad
parent015f416b5cc353c50ba9bc8ec3d9227d08f7b920
app_sms: Fix uninitialized values; hangup channel when REL is sent successfully

This patch fixes two issues in app_sms:
(1) Firstly, the 'flags' field on the stack in sms_exec() is uninitialised,
    causing it to use the wrong protocol in some cases. This patch correctly
    initializes the flags fields.

(2) Secondly, when disconnect supervision is not working or
    inbanddisconnect=yes is set in chan_dahdi.conf, app_sms was failing to
    terminate the call after it sent the REL(ease) message and the peer stopped
    talking to it. This patch fixes the code to handle the 'bad stop bit'
    message more gracefully in that case, and hang up the call.

Review: https://reviewboard.asterisk.org/r/1392/

ASTERISK-18331 #close
Reported by: David Woodhouse
patches:
  asterisk-fix-sms.patch uploaded by David Woodhouse (License 5754)

git-svn-id: http://svn.asterisk.org/svn/asterisk/branches/1.8@412655 f38db490-d61c-443f-a65b-d21fe96a405b
apps/app_sms.c