Callerid fix and warning cleanup (bug #367 & #356)
authorMark Spencer <markster@digium.com>
Wed, 8 Oct 2003 21:57:43 +0000 (21:57 +0000)
committerMark Spencer <markster@digium.com>
Wed, 8 Oct 2003 21:57:43 +0000 (21:57 +0000)
git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@1621 65c4cc65-6c06-0410-ace0-fbb531ad65f3

callerid.c
channels/chan_zap.c
frame.c

index 0a0883e..3766f4b 100755 (executable)
@@ -110,8 +110,8 @@ struct callerid_state *callerid_new(void)
 {
        struct callerid_state *cid;
        cid = malloc(sizeof(struct callerid_state));
 {
        struct callerid_state *cid;
        cid = malloc(sizeof(struct callerid_state));
-       memset(cid, 0, sizeof(struct callerid_state));
        if (cid) {
        if (cid) {
+               memset(cid, 0, sizeof(struct callerid_state));
                cid->fskd.spb = 7;              /* 1200 baud */
                cid->fskd.hdlc = 0;             /* Async */
                cid->fskd.nbit = 8;             /* 8 bits */
                cid->fskd.spb = 7;              /* 1200 baud */
                cid->fskd.hdlc = 0;             /* Async */
                cid->fskd.nbit = 8;             /* 8 bits */
index 723c466..75bead1 100755 (executable)
@@ -4359,6 +4359,8 @@ static void *ss_thread(void *data)
                        } else
                                ast_log(LOG_WARNING, "Unable to get caller ID space\n");
                }
                        } else
                                ast_log(LOG_WARNING, "Unable to get caller ID space\n");
                }
+               else
+                       cs = NULL;
                if (name && number) {
                        snprintf(cid, sizeof(cid), "\"%s\" <%s>", name, number);
                } else if (name) {
                if (name && number) {
                        snprintf(cid, sizeof(cid), "\"%s\" <%s>", name, number);
                } else if (name) {
@@ -4368,6 +4370,8 @@ static void *ss_thread(void *data)
                } else {
                        strcpy(cid, "");
                }
                } else {
                        strcpy(cid, "");
                }
+               if (cs)
+                       callerid_free(cs);
                if (strlen(cid)) {
                        chan->callerid = strdup(cid);
                        chan->ani = strdup(cid);
                if (strlen(cid)) {
                        chan->callerid = strdup(cid);
                        chan->ani = strdup(cid);
diff --git a/frame.c b/frame.c
index 36ac0e1..0ff9d2a 100755 (executable)
--- a/frame.c
+++ b/frame.c
@@ -455,8 +455,7 @@ static int show_codecs(int fd, int argc, char *argv[])
                return RESULT_SHOWUSAGE;
 
        if (getenv("I_AM_NOT_AN_IDIOT") == NULL)
                return RESULT_SHOWUSAGE;
 
        if (getenv("I_AM_NOT_AN_IDIOT") == NULL)
-               ast_cli(fd, "Disclaimer: this command is for informational purposes
-only.\n\tIt does not indicate anything about your configuration.\n");
+               ast_cli(fd, "Disclaimer: this command is for informational purposes only.\n\tIt does not indicate anything about your configuration.\n");
 
        if ((argc == 2) || (!strcasecmp(argv[1],"audio"))) {
                found = 1;
 
        if ((argc == 2) || (!strcasecmp(argv[1],"audio"))) {
                found = 1;