diff options
| author | Mark Michelson <mmichelson@digium.com> | 2012-05-18 17:54:07 +0000 |
|---|---|---|
| committer | Mark Michelson <mmichelson@digium.com> | 2012-05-18 17:54:07 +0000 |
| commit | 11348736af2097be6ca9d99f160ead3e85e44ec2 (patch) | |
| tree | bae82375700dbfa7752f22159d17deb8d6f1315a /channels/chan_dahdi.c | |
| parent | 5c576aa3c2bfd5b8cb13edff88692c7274d906af (diff) | |
Address MISSING_BREAK static analysis reports some more.
This addresses core findings 4 and 6.
Moises Silva helped me by stating that a break could be
safely added to the case where it is added in chan_dahdi.c
In say.c, I have added a comment indicating that static analysis
complains but that it is currently unknown if this is correct.
This fixes all core findings of this type.
(closes issue ASTERISK-19662)
reported by Matthew Jordan
........
Merged revisions 367027 from http://svn.asterisk.org/svn/asterisk/branches/1.8
........
Merged revisions 367028 from http://svn.asterisk.org/svn/asterisk/branches/10
git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@367029 65c4cc65-6c06-0410-ace0-fbb531ad65f3
Diffstat (limited to 'channels/chan_dahdi.c')
| -rw-r--r-- | channels/chan_dahdi.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/channels/chan_dahdi.c b/channels/chan_dahdi.c index 38a894395..0db84df1f 100644 --- a/channels/chan_dahdi.c +++ b/channels/chan_dahdi.c @@ -8051,6 +8051,7 @@ static struct ast_frame *dahdi_handle_event(struct ast_channel *ast) #else ast_log(LOG_WARNING, "Received bits changed on %s signalling?\n", sig2str(p->sig)); #endif + break; case DAHDI_EVENT_PULSE_START: /* Stop tone if there's a pulse start and the PBX isn't started */ if (!ast_channel_pbx(ast)) |