Asterisk
/
asterisk
mirror of https://github.com/asterisk/asterisk
1
0
Fork 0
Code Issues Releases Wiki Activity

chan_console: Fix reference leaks to pvt.

Browse Source 
Fix a bunch of calls to get_active_pvt
where the reference is never released.

ASTERISK-24504 #close
Reported by: Corey Farrell
Review: https://reviewboard.asterisk.org/r/4152/


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/11@427554 65c4cc65-6c06-0410-ace0-fbb531ad65f3
changes/61/61/1
Corey Farrell 11 years ago
parent 8745e12323
commit ed7dabef46
1 changed files with 40 additions and 24 deletions
Show Stats Download Patch File Download Diff File

64
channels/chan_console.c
Unescape View File

@ -687,7 +687,7 @@ static struct console_pvt *get_active_pvt(void)
static char *cli_console_autoanswer(struct ast_cli_entry *e, int cmd,
struct ast_cli_args *a)
{
struct console_pvt *pvt = get_active_pvt();
struct console_pvt *pvt;
char *res = CLI_SUCCESS;
switch (cmd) {
@ -704,6 +704,7 @@ static char *cli_console_autoanswer(struct ast_cli_entry *e, int cmd,
return NULL;
}
pvt = get_active_pvt();
if (!pvt) {
ast_cli(a->fd, "No console device is set as active.\n");
return CLI_FAILURE;
@ -734,7 +735,7 @@ static char *cli_console_autoanswer(struct ast_cli_entry *e, int cmd,
static char *cli_console_flash(struct ast_cli_entry *e, int cmd, struct ast_cli_args *a)
{
struct console_pvt *pvt = get_active_pvt();
struct console_pvt *pvt;
if (cmd == CLI_INIT) {
e->command = "console flash";
@ -742,17 +743,20 @@ static char *cli_console_flash(struct ast_cli_entry *e, int cmd, struct ast_cli_
"Usage: console flash\n"
" Flashes the call currently placed on the console.\n";
return NULL;
} else if (cmd == CLI_GENERATE)
} else if (cmd == CLI_GENERATE) {
return NULL;
}
if (a->argc != e->args) {
return CLI_SHOWUSAGE;
}
pvt = get_active_pvt();
if (!pvt) {
ast_cli(a->fd, "No console device is set as active\n");
return CLI_FAILURE;
}
if (a->argc != e->args)
return CLI_SHOWUSAGE;
if (!pvt->owner) {
ast_cli(a->fd, "No call to flash\n");
unref_pvt(pvt);
@ -772,7 +776,7 @@ static char *cli_console_dial(struct ast_cli_entry *e, int cmd, struct ast_cli_a
{
char *s = NULL;
const char *mye = NULL, *myc = NULL;
struct console_pvt *pvt = get_active_pvt();
struct console_pvt *pvt;
if (cmd == CLI_INIT) {
e->command = "console dial";
@ -780,16 +784,19 @@ static char *cli_console_dial(struct ast_cli_entry *e, int cmd, struct ast_cli_a
"Usage: console dial [extension[@context]]\n"
" Dials a given extension (and context if specified)\n";
return NULL;
} else if (cmd == CLI_GENERATE)
} else if (cmd == CLI_GENERATE) {
return NULL;
}
if (a->argc > e->args + 1) {
return CLI_SHOWUSAGE;
}
pvt = get_active_pvt();
if (!pvt) {
ast_cli(a->fd, "No console device is currently set as active\n");
return CLI_FAILURE;
}
if (a->argc > e->args + 1)
return CLI_SHOWUSAGE;
if (pvt->owner) { /* already in a call */
int i;
@ -844,7 +851,7 @@ static char *cli_console_dial(struct ast_cli_entry *e, int cmd, struct ast_cli_a
static char *cli_console_hangup(struct ast_cli_entry *e, int cmd, struct ast_cli_args *a)
{
struct console_pvt *pvt = get_active_pvt();
struct console_pvt *pvt;
if (cmd == CLI_INIT) {
e->command = "console hangup";
@ -852,16 +859,19 @@ static char *cli_console_hangup(struct ast_cli_entry *e, int cmd, struct ast_cli
"Usage: console hangup\n"
" Hangs up any call currently placed on the console.\n";
return NULL;
} else if (cmd == CLI_GENERATE)
} else if (cmd == CLI_GENERATE) {
return NULL;
}
if (a->argc != e->args) {
return CLI_SHOWUSAGE;
}
pvt = get_active_pvt();
if (!pvt) {
ast_cli(a->fd, "No console device is set as active\n");
return CLI_FAILURE;
}
if (a->argc != e->args)
return CLI_SHOWUSAGE;
if (!pvt->owner && !pvt->hookstate) {
ast_cli(a->fd, "No call to hang up\n");
@ -881,7 +891,7 @@ static char *cli_console_hangup(struct ast_cli_entry *e, int cmd, struct ast_cli
static char *cli_console_mute(struct ast_cli_entry *e, int cmd, struct ast_cli_args *a)
{
const char *s;
struct console_pvt *pvt = get_active_pvt();
struct console_pvt *pvt;
char *res = CLI_SUCCESS;
if (cmd == CLI_INIT) {
@ -890,17 +900,20 @@ static char *cli_console_mute(struct ast_cli_entry *e, int cmd, struct ast_cli_a
"Usage: console {mute|unmute}\n"
" Mute/unmute the microphone.\n";
return NULL;
} else if (cmd == CLI_GENERATE)
} else if (cmd == CLI_GENERATE) {
return NULL;
}
if (a->argc != e->args) {
return CLI_SHOWUSAGE;
}
pvt = get_active_pvt();
if (!pvt) {
ast_cli(a->fd, "No console device is set as active\n");
return CLI_FAILURE;
}
if (a->argc != e->args)
return CLI_SHOWUSAGE;
s = a->argv[e->args-1];
if (!strcasecmp(s, "mute"))
pvt->muted = 1;
@ -1028,7 +1041,7 @@ static char *cli_list_devices(struct ast_cli_entry *e, int cmd, struct ast_cli_a
*/
static char *cli_console_answer(struct ast_cli_entry *e, int cmd, struct ast_cli_args *a)
{
struct console_pvt *pvt = get_active_pvt();
struct console_pvt *pvt;
switch (cmd) {
case CLI_INIT:
@ -1042,6 +1055,7 @@ static char *cli_console_answer(struct ast_cli_entry *e, int cmd, struct ast_cli
return NULL; /* no completion */
}
pvt = get_active_pvt();
if (!pvt) {
ast_cli(a->fd, "No console device is set as active\n");
return CLI_FAILURE;
@ -1078,7 +1092,7 @@ static char *cli_console_answer(struct ast_cli_entry *e, int cmd, struct ast_cli
static char *cli_console_sendtext(struct ast_cli_entry *e, int cmd, struct ast_cli_args *a)
{
char buf[TEXT_SIZE];
struct console_pvt *pvt = get_active_pvt();
struct console_pvt *pvt;
struct ast_frame f = {
.frametype = AST_FRAME_TEXT,
.data.ptr = buf,
@ -1092,9 +1106,11 @@ static char *cli_console_sendtext(struct ast_cli_entry *e, int cmd, struct ast_c
"Usage: console send text <message>\n"
" Sends a text message for display on the remote terminal.\n";
return NULL;
} else if (cmd == CLI_GENERATE)
} else if (cmd == CLI_GENERATE) {
return NULL;
}
pvt = get_active_pvt();
if (!pvt) {
ast_cli(a->fd, "No console device is set as active\n");
return CLI_FAILURE;

Write Preview
Loading…
Cancel
Save