diff options
author | rizzo <rizzo@f38db490-d61c-443f-a65b-d21fe96a405b> | 2006-04-19 06:58:20 +0000 |
---|---|---|
committer | rizzo <rizzo@f38db490-d61c-443f-a65b-d21fe96a405b> | 2006-04-19 06:58:20 +0000 |
commit | e05d203399a4f8131da7d4f1fa31ef3fae529866 (patch) | |
tree | df65fdd7ba71ef4f06e025db6b61645d92e2cb03 /channel.c | |
parent | 053d1ead69adb76e47bd8008324f25461c218831 (diff) |
- use ast_calloc instead of ast_malloc
- use ast_channel_lock/unlock in a few places
- comment some dubious pieces of code
- use memset to zero a buffer
git-svn-id: http://svn.digium.com/svn/asterisk/trunk@21311 f38db490-d61c-443f-a65b-d21fe96a405b
Diffstat (limited to 'channel.c')
-rw-r--r-- | channel.c | 17 |
1 files changed, 6 insertions, 11 deletions
@@ -401,7 +401,7 @@ int ast_channel_register(const struct ast_channel_tech *tech) } } - if (!(chan = ast_malloc(sizeof(*chan)))) { + if (!(chan = ast_calloc(1, sizeof(*chan)))) { AST_LIST_UNLOCK(&channels); return -1; } @@ -2083,15 +2083,15 @@ int ast_indicate(struct ast_channel *chan, int condition) { int res = -1; - ast_mutex_lock(&chan->lock); + ast_channel_lock(chan); /* Stop if we're a zombie or need a soft hangup */ if (ast_test_flag(chan, AST_FLAG_ZOMBIE) || ast_check_hangup(chan)) { - ast_mutex_unlock(&chan->lock); + ast_channel_unlock(chan); return -1; } if (chan->tech->indicate) res = chan->tech->indicate(chan, condition); - ast_mutex_unlock(&chan->lock); + ast_channel_unlock(chan); if (!chan->tech->indicate || res) { /* * Device does not support (that) indication, lets fake @@ -2333,7 +2333,7 @@ int ast_write(struct ast_channel *chan, struct ast_frame *fr) break; case AST_FRAME_VOICE: if (chan->tech->write == NULL) - break; + break; /*! \todo XXX should return 0 maybe ? */ /* Bypass translator if we're writing format in the raw write format. This allows mixing of native / non-native formats */ @@ -4014,7 +4014,6 @@ static void silence_generator_release(struct ast_channel *chan, void *data) static int silence_generator_generate(struct ast_channel *chan, void *data, int len, int samples) { short buf[samples]; - int x; struct ast_frame frame = { .frametype = AST_FRAME_VOICE, .subclass = AST_FORMAT_SLINEAR, @@ -4022,13 +4021,9 @@ static int silence_generator_generate(struct ast_channel *chan, void *data, int .samples = samples, .datalen = sizeof(buf), }; - - for (x = 0; x < samples; x++) - buf[x] = 0; - + memset(buf, 0, sizeof(buf)); if (ast_write(chan, &frame)) return -1; - return 0; } |