diff options
author | tilghman <tilghman@f38db490-d61c-443f-a65b-d21fe96a405b> | 2008-06-11 18:18:16 +0000 |
---|---|---|
committer | tilghman <tilghman@f38db490-d61c-443f-a65b-d21fe96a405b> | 2008-06-11 18:18:16 +0000 |
commit | 02e87fc6524c2df91cfa8662689f3b33985e3eae (patch) | |
tree | f88b7951408fd32dee36fdba67ee16e72e554a5f | |
parent | 178171468aa29b67846684a743c388015193141c (diff) |
Make calls to ast_assert() actually test something, so that the error message
printed is not nonsensical (reported by mvanbaak via #asterisk-bugs).
git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@121861 f38db490-d61c-443f-a65b-d21fe96a405b
-rw-r--r-- | channels/chan_agent.c | 2 | ||||
-rw-r--r-- | main/abstract_jb.c | 9 | ||||
-rw-r--r-- | main/channel.c | 4 | ||||
-rw-r--r-- | main/sched.c | 2 |
4 files changed, 8 insertions, 9 deletions
diff --git a/channels/chan_agent.c b/channels/chan_agent.c index f216030cf..0f6fbfbe8 100644 --- a/channels/chan_agent.c +++ b/channels/chan_agent.c @@ -1061,7 +1061,7 @@ static struct ast_channel *agent_new(struct agent_pvt *p, int state) if (p->chan) { if (ast_test_flag(p->chan, AST_FLAG_BLOCKING)) { ast_log( LOG_ERROR, "A blocker exists after agent channel ownership acquired\n" ); - ast_assert(0); + ast_assert(ast_test_flag(p->chan, AST_FLAG_BLOCKING) == 0); } } return tmp; diff --git a/main/abstract_jb.c b/main/abstract_jb.c index 25bd8fd0d..1fd4dcdcd 100644 --- a/main/abstract_jb.c +++ b/main/abstract_jb.c @@ -433,7 +433,7 @@ static void jb_get_and_deliver(struct ast_channel *chan) return; default: ast_log(LOG_ERROR, "This should never happen!\n"); - ast_assert(0); + ast_assert("JB type unknown" == NULL); break; } @@ -487,10 +487,9 @@ static int create_jb(struct ast_channel *chan, struct ast_frame *frr) *tmp = '#'; bridged = ast_bridged_channel(chan); - if (!bridged) { - /* We should always have bridged chan if a jitterbuffer is in use */ - ast_assert(0); - } + /* We should always have bridged chan if a jitterbuffer is in use */ + ast_assert(bridged != NULL); + snprintf(name1, sizeof(name1), "%s", bridged->name); tmp = strchr(name1, '/'); if (tmp) diff --git a/main/channel.c b/main/channel.c index 5fff0f7d9..ad7513cff 100644 --- a/main/channel.c +++ b/main/channel.c @@ -912,7 +912,7 @@ int ast_queue_frame(struct ast_channel *chan, struct ast_frame *fin) if (((fin->frametype == AST_FRAME_VOICE) && (qlen > 96)) || (qlen > 128)) { if (fin->frametype != AST_FRAME_VOICE) { ast_log(LOG_WARNING, "Exceptionally long queue length queuing to %s\n", chan->name); - ast_assert(0); + ast_assert(fin->frametype == AST_FRAME_VOICE); } else { if (option_debug) ast_log(LOG_DEBUG, "Dropping voice to exceptionally long queue on %s\n", chan->name); @@ -1476,7 +1476,7 @@ int ast_hangup(struct ast_channel *chan) ast_log(LOG_WARNING, "Hard hangup called by thread %ld on %s, while fd " "is blocked by thread %ld in procedure %s! Expect a failure\n", (long)pthread_self(), chan->name, (long)chan->blocker, chan->blockproc); - ast_assert(0); + ast_assert(ast_test_flag(chan, AST_FLAG_BLOCKING) == 0); } if (!ast_test_flag(chan, AST_FLAG_ZOMBIE)) { if (option_debug) diff --git a/main/sched.c b/main/sched.c index 935e85398..f020aa196 100644 --- a/main/sched.c +++ b/main/sched.c @@ -282,7 +282,7 @@ int ast_sched_del(struct sched_context *con, int id) if (!s) { if (option_debug) ast_log(LOG_DEBUG, "Attempted to delete nonexistent schedule entry %d!\n", id); - ast_assert(0); + ast_assert(s != NULL); return -1; } |