Put in missing \ns on the end of ast_logs (issue #7936 reported by wojtekka)

git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@43933 65c4cc65-6c06-0410-ace0-fbb531ad65f3

diff --git a/apps/app_alarmreceiver.c b/apps/app_alarmreceiver.c
index d2741df93fc8873f75c471525161cc670029838a..8afce25d577b80c515b7a0a8a730d405b8e9253d 100644 (file)
--- a/apps/app_alarmreceiver.c
+++ b/apps/app_alarmreceiver.c
@@ -136,7 +136,7 @@ static void database_increment( char *key )
        res = ast_db_put(db_family, key, value);
        
        if((res)&&(option_verbose >= 4))
-               ast_verbose(VERBOSE_PREFIX_4 "AlarmReceiver: database_increment write error");
+               ast_verbose(VERBOSE_PREFIX_4 "AlarmReceiver: database_increment write error\n");
        
        return; 
 }

diff --git a/apps/app_festival.c b/apps/app_festival.c
index cf0dfcba4f457fd4d2b0a5e9209b6d833aff49b8..6e7a4ffd49de857da49940c19d800effdd2e7502 100644 (file)
--- a/apps/app_festival.c
+++ b/apps/app_festival.c
@@ -485,7 +485,7 @@ static int festival_exec(struct ast_channel *chan, void *vdata)
                         * */
                        if ( read_data == -1 )
                        {
-                               ast_log(LOG_WARNING,"Unable to read from cache/festival fd");
+                               ast_log(LOG_WARNING,"Unable to read from cache/festival fd\n");
                               close(fd);
                               ast_config_destroy(cfg);
                               ast_module_user_remove(u);

diff --git a/apps/app_followme.c b/apps/app_followme.c
index 0ee3abc3a0558125fab2644aa11a37ee36f3d525..f83cfa1657f18e25d7b10e9cc1fe8df9ed0038c9 100644 (file)
--- a/apps/app_followme.c
+++ b/apps/app_followme.c
@@ -951,7 +951,7 @@ static int app_exec(struct ast_channel *chan, void *data)
        if (option_debug)
                ast_log(LOG_DEBUG, "New profile %s.\n", args.followmeid);
        if (!f) { 
-               ast_log(LOG_WARNING, "Profile requested, %s, not found in the configuration.", args.followmeid);
+               ast_log(LOG_WARNING, "Profile requested, %s, not found in the configuration.\n", args.followmeid);
                res = -1;
        } else {
                /* XXX TODO: Reinsert the db check value to see whether or not follow-me is on or off */

diff --git a/apps/app_playback.c b/apps/app_playback.c
index 396ed71e6cf34e7425f14c777a8c60ecfe85c453..5dfb589f712a6536e6f00c4044c39d693d862413 100644 (file)
--- a/apps/app_playback.c
+++ b/apps/app_playback.c
@@ -341,7 +341,7 @@ static int __say_init(int fd, int argc, char *argv[])
                 return RESULT_SHOWUSAGE;
         mode = argv[2];
 
-       ast_log(LOG_WARNING, "init say.c from %s to %s", old_mode, mode);
+       ast_log(LOG_WARNING, "init say.c from %s to %s\n", old_mode, mode);
 
        if (!strcmp(mode, old_mode)) {
                ast_log(LOG_WARNING, "say mode is %s already\n", mode);

diff --git a/apps/app_rpt.c b/apps/app_rpt.c
index 305e59eb4a92d8ac2139aa3ad5292a7358f66d89..ac966e9bbb522e847745dacc49a65a6858f0c237 100644 (file)
--- a/apps/app_rpt.c
+++ b/apps/app_rpt.c
@@ -2618,7 +2618,7 @@ pthread_attr_t attr;
                rpt_mutex_lock(&myrpt->lock);
                remque((struct qlem *) tele); /* We don't like stuck transmitters, remove it from the queue */
                rpt_mutex_unlock(&myrpt->lock); 
-               ast_log(LOG_WARNING, "Could not create telemetry thread: %s",strerror(res));
+               ast_log(LOG_WARNING, "Could not create telemetry thread: %s\n",strerror(res));
        }
        return;
 }

diff --git a/channels/chan_iax2.c b/channels/chan_iax2.c
index 9aceac5a7bdf5d8f06c6e02cbd994f90a045662d..cf99ea803c0254b90db49acc32f6fdb085eed687 100644 (file)
--- a/channels/chan_iax2.c
+++ b/channels/chan_iax2.c
@@ -5425,7 +5425,7 @@ static int iax2_register(char *value, int lineno)
        username = strsep(&stringp, "@");
        hostname = strsep(&stringp, "@");
        if (!hostname) {
-               ast_log(LOG_WARNING, "Format for registration is user[:secret]@host[:port] at line %d", lineno);
+               ast_log(LOG_WARNING, "Format for registration is user[:secret]@host[:port] at line %d\n", lineno);
                return -1;
        }
        stringp=username;

diff --git a/channels/chan_mgcp.c b/channels/chan_mgcp.c
index 697561784349662ceb598145a2417293527f7faf..b2d0e109052bc2f31370552df619357b3cecdf48 100644 (file)
--- a/channels/chan_mgcp.c
+++ b/channels/chan_mgcp.c
@@ -3004,7 +3004,7 @@ static void handle_hd_hf(struct mgcp_subchannel *sub, char *ev)
                                ast_log(LOG_WARNING, "Off hook, but already have owner on %s@%s\n", p->name, p->parent->name);
                        } else {
                                ast_log(LOG_WARNING, "On hook, but already have owner on %s@%s\n", p->name, p->parent->name);
-                               ast_log(LOG_WARNING, "If we're onhook why are we here trying to handle a hd or hf?");
+                               ast_log(LOG_WARNING, "If we're onhook why are we here trying to handle a hd or hf?\n");
                        }
                        if (ast_bridged_channel(sub->owner))
                                ast_queue_control(sub->owner, AST_CONTROL_UNHOLD);
@@ -3793,7 +3793,7 @@ static struct mgcp_gateway *build_gateway(char *cat, struct ast_variable *v)
                                                                e->sub = sub;
                                                        } else {
                                                                /* XXX Should find a way to clean up our memory */
-                                                               ast_log(LOG_WARNING, "Out of memory allocating subchannel");
+                                                               ast_log(LOG_WARNING, "Out of memory allocating subchannel\n");
                                                                return NULL;
                                                        }
                                                }
@@ -3910,7 +3910,7 @@ static struct mgcp_gateway *build_gateway(char *cat, struct ast_variable *v)
                                                        sub->nat = nat;
                                                } else {
                                                        /* XXX Should find a way to clean up our memory */
-                                                       ast_log(LOG_WARNING, "Out of memory allocating subchannel");
+                                                       ast_log(LOG_WARNING, "Out of memory allocating subchannel\n");
                                                        return NULL;
                                                }
                                        }

diff --git a/channels/chan_misdn.c b/channels/chan_misdn.c
index e6ac85079115b1c9a440d31e4836c0f359ff74b7..c1a0f0dfc4883cf2f81ade3f851c67b574abb9d1 100644 (file)
--- a/channels/chan_misdn.c
+++ b/channels/chan_misdn.c
@@ -1896,7 +1896,7 @@ static int misdn_call(struct ast_channel *ast, char *dest, int timeout)
        
        {
                struct chan_list *ch=MISDN_ASTERISK_TECH_PVT(ast);
-               if (!ch) { ast_verbose("No chan_list in misdn_call"); return -1;}
+               if (!ch) { ast_verbose("No chan_list in misdn_call\n"); return -1;}
                
                newbc->capability=ast->transfercapability;
                pbx_builtin_setvar_helper(ast,"TRANSFERCAPABILITY",ast_transfercapability2str(newbc->capability));
@@ -2392,7 +2392,7 @@ static struct ast_frame *process_ast_dsp(struct chan_list *tmp, struct ast_frame
        if (!f || (f->frametype != AST_FRAME_DTMF))
                return frame;
  
-       ast_log(LOG_DEBUG, "Detected inband DTMF digit: %c", f->subclass);
+       ast_log(LOG_DEBUG, "Detected inband DTMF digit: %c\n", f->subclass);
  
        if (tmp->faxdetect && (f->subclass == 'f')) {
                /* Fax tone -- Handle and return NULL */

diff --git a/channels/chan_sip.c b/channels/chan_sip.c
index a1d30257a1a617006003d03d08a7edd3e8a6aabf..7030741e8cd3810212b8faca2dcfc977caf6cb63 100644 (file)
--- a/channels/chan_sip.c
+++ b/channels/chan_sip.c
@@ -3987,7 +3987,7 @@ static struct ast_frame *sip_read(struct ast_channel *ast)
                                p->t38.state = T38_LOCAL_REINVITE;
                                transmit_reinvite_with_t38_sdp(p);
                                if (option_debug > 1)
-                                       ast_log(LOG_DEBUG, "T38 state changed to %d on channel %s", p->t38.state, ast->name);
+                                       ast_log(LOG_DEBUG, "T38 state changed to %d on channel %s\n", p->t38.state, ast->name);
                        }
                } else if (!ast_test_flag(&p->flags[0], SIP_PENDINGBYE)) {
                        if (option_debug > 2)
@@ -14307,10 +14307,7 @@ static int sipsock_read(int *id, int fd, short events, void *ignore)
        parse_request(&req);
        req.method = find_sip_method(req.rlPart1);
        if (ast_test_flag(&req, SIP_PKT_DEBUG)) {
-               ast_verbose("--- (%d headers %d lines)", req.headers, req.lines);
-               if (req.headers + req.lines == 0) 
-                       ast_verbose(" Nat keepalive ");
-               ast_verbose("---\n");
+               ast_verbose("--- (%d headers %d lines)%s ---\n", req.headers, req.lines, (req.headers + req.lines == 0) ? " Nat keepalive" : "");
        }
 
        if (req.headers < 2) {

diff --git a/channels/chan_skinny.c b/channels/chan_skinny.c
index fe839aa486b574b4a93b2f64480e1a8fdb3f5e73..453380881a31383c4e74a01b43ef5d529a1a341e 100644 (file)
--- a/channels/chan_skinny.c
+++ b/channels/chan_skinny.c
@@ -4507,7 +4507,7 @@ static int reload_config(void)
        if (skinnysock < 0) {
                skinnysock = socket(AF_INET, SOCK_STREAM, 0);
                if(setsockopt(skinnysock, SOL_SOCKET, SO_REUSEADDR, &on, sizeof(on)) == -1) {
-                       ast_log(LOG_ERROR, "Set Socket Options failed: errno %d, %s", errno, strerror(errno));
+                       ast_log(LOG_ERROR, "Set Socket Options failed: errno %d, %s\n", errno, strerror(errno));
                        ast_config_destroy(cfg);
                        return 0;
                }

diff --git a/channels/iax2-provision.c b/channels/iax2-provision.c
index c2c4c88c66a610818817478ed0d67922c02365dd..b6137a88b51cbaf7564b6c015f84bd0998f84a04 100644 (file)
--- a/channels/iax2-provision.c
+++ b/channels/iax2-provision.c
@@ -283,7 +283,7 @@ static int iax_template_parse(struct iax_template *cur, struct ast_config *cfg,
                if (!src) {
                        src = iax_template_find(def, 0);
                        if (!src)
-                               ast_log(LOG_WARNING, "Unable to locate default base template '%s' for creating '%s', omitting.", def, s);
+                               ast_log(LOG_WARNING, "Unable to locate default base template '%s' for creating '%s', omitting.\n", def, s);
                }
                if (!src)
                        return -1;

diff --git a/funcs/func_timeout.c b/funcs/func_timeout.c
index 206ba948253a69edc115f39267e5666645f2886b..b88ae078f53b7345ffff6eeb341328e8027d9622 100644 (file)
--- a/funcs/func_timeout.c
+++ b/funcs/func_timeout.c
@@ -46,7 +46,7 @@ static int timeout_read(struct ast_channel *chan, char *cmd, char *data,
        time_t myt;
 
        if (!data) {
-               ast_log(LOG_ERROR, "Must specify type of timeout to get.");
+               ast_log(LOG_ERROR, "Must specify type of timeout to get.\n");
                return -1;
        }
 
@@ -76,7 +76,7 @@ static int timeout_read(struct ast_channel *chan, char *cmd, char *data,
                break;
 
        default:
-               ast_log(LOG_ERROR, "Unknown timeout type specified.");
+               ast_log(LOG_ERROR, "Unknown timeout type specified.\n");
                break;
        }
 
@@ -91,7 +91,7 @@ static int timeout_write(struct ast_channel *chan, char *cmd, char *data,
        struct tm myt;
 
        if (!data) {
-               ast_log(LOG_ERROR, "Must specify type of timeout to set.");
+               ast_log(LOG_ERROR, "Must specify type of timeout to set.\n");
                return -1;
        }
 
@@ -137,7 +137,7 @@ static int timeout_write(struct ast_channel *chan, char *cmd, char *data,
                break;
 
        default:
-               ast_log(LOG_ERROR, "Unknown timeout type specified.");
+               ast_log(LOG_ERROR, "Unknown timeout type specified.\n");
                break;
        }
 

diff --git a/include/asterisk/logger.h b/include/asterisk/logger.h
index 6e486620df8a5c3ffad0d6bc427b80558d0db92c..b4737d06cfddd93a139251a2ba9ee299a271d7dd 100644 (file)
--- a/include/asterisk/logger.h
+++ b/include/asterisk/logger.h
@@ -48,7 +48,7 @@ extern "C" {
 /*! Used for sending a log message */
 /*!
        \brief This is the standard logger function.  Probably the only way you will invoke it would be something like this:
-       ast_log(LOG_WHATEVER, "Problem with the %s Captain.  We should get some more.  Will %d be enough?", "flux capacitor", 10);
+       ast_log(LOG_WHATEVER, "Problem with the %s Captain.  We should get some more.  Will %d be enough?\n", "flux capacitor", 10);
        where WHATEVER is one of ERROR, DEBUG, EVENT, NOTICE, or WARNING depending
        on which log you wish to output to. These are implemented as macros, that
        will provide the function with the needed arguments.

diff --git a/main/frame.c b/main/frame.c
index c5ddad8d1c4b086af140e205cb8f590b959b8512..4bbbcbdae79cc86ec56cc9e85648ebe315ad5aa6 100644 (file)
--- a/main/frame.c
+++ b/main/frame.c
@@ -229,7 +229,7 @@ struct ast_frame *ast_smoother_read(struct ast_smoother *s)
        /* IF we have an optimization frame, send it */
        if (s->opt) {
                if (s->opt->offset < AST_FRIENDLY_OFFSET)
-                       ast_log(LOG_WARNING, "Returning a frame of inappropriate offset (%d).",
+                       ast_log(LOG_WARNING, "Returning a frame of inappropriate offset (%d).\n",
                                                        s->opt->offset);
                opt = s->opt;
                s->opt = NULL;

diff --git a/main/udptl.c b/main/udptl.c
index 51d1fb97b600fbeebaa4743d5c5050cd444c2dd5..bd3a3acb46ad343aa0eb9a9816b506b56a066947 100644 (file)
--- a/main/udptl.c
+++ b/main/udptl.c
@@ -712,7 +712,7 @@ void ast_udptl_set_error_correction_scheme(struct ast_udptl* udptl, int ec)
                        udptl->error_correction_scheme = UDPTL_ERROR_CORRECTION_NONE;
                        break;
                default:
-                       ast_log(LOG_WARNING, "error correction parameter invalid");
+                       ast_log(LOG_WARNING, "error correction parameter invalid\n");
                };
        } else
                ast_log(LOG_WARNING, "udptl structure is null\n");

diff --git a/res/res_monitor.c b/res/res_monitor.c
index e81325da7d5102c45e92a9860ddeb6dd1f597c86..ebac350f80cb3faaf8318b30ee12e714da58e42a 100644 (file)
--- a/res/res_monitor.c
+++ b/res/res_monitor.c
@@ -338,7 +338,7 @@ int ast_monitor_change_fname(struct ast_channel *chan, const char *fname_base, i
 {
        char tmp[256];
        if (ast_strlen_zero(fname_base)) {
-               ast_log(LOG_WARNING, "Cannot change monitor filename of channel %s to null", chan->name);
+               ast_log(LOG_WARNING, "Cannot change monitor filename of channel %s to null\n", chan->name);
                return -1;
        }
 

diff --git a/res/res_musiconhold.c b/res/res_musiconhold.c
index 13053306961342a8106ddab2eb196310822680db..19d0fd0690efc76db0d7d7ffef65e3a107e69e7e 100644 (file)
--- a/res/res_musiconhold.c
+++ b/res/res_musiconhold.c
@@ -1245,7 +1245,7 @@ static int load_module(void)
                res = ast_register_application(app4, moh4_exec, synopsis4, descrip4);
 
        if (!init_classes(0)) {         /* No music classes configured, so skip it */
-               ast_log(LOG_WARNING, "No music on hold classes configured, disabling music on hold.");
+               ast_log(LOG_WARNING, "No music on hold classes configured, disabling music on hold.\n");
        } else {
                ast_install_music_functions(local_ast_moh_start, local_ast_moh_stop, local_ast_moh_cleanup);
        }

diff --git a/res/res_smdi.c b/res/res_smdi.c
index c939aba27479bbb913e58e5636a7ad7c3b40eff8..c0c7f8927755638a47b74a0299fbe59f1987c3d6 100644 (file)
--- a/res/res_smdi.c
+++ b/res/res_smdi.c
@@ -143,7 +143,7 @@ int ast_smdi_mwi_unset(struct ast_smdi_interface *iface, const char *mailbox)
        fclose(file);
 
        ASTOBJ_UNLOCK(iface);
-       ast_log(LOG_DEBUG, "Sent MWI unset message for %s on %s", mailbox, iface->name);
+       ast_log(LOG_DEBUG, "Sent MWI unset message for %s on %s\n", mailbox, iface->name);
        return 0;
 }
 
@@ -199,7 +199,7 @@ struct ast_smdi_md_message *ast_smdi_md_message_pop(struct ast_smdi_interface *i
                if (elapsed > iface->msg_expiry) {
                        /* found an expired message */
                        ASTOBJ_UNREF(md_msg, ast_smdi_md_message_destroy);
-                       ast_log(LOG_NOTICE, "Purged expired message from %s SMDI MD message queue.  Message was %ld milliseconds too old.",
+                       ast_log(LOG_NOTICE, "Purged expired message from %s SMDI MD message queue.  Message was %ld milliseconds too old.\n",
                                iface->name, elapsed - iface->msg_expiry);
                        md_msg = ASTOBJ_CONTAINER_UNLINK_START(&iface->md_q);
                }
@@ -267,7 +267,7 @@ extern struct ast_smdi_mwi_message *ast_smdi_mwi_message_pop(struct ast_smdi_int
                if (elapsed > iface->msg_expiry) {
                        /* found an expired message */
                        ASTOBJ_UNREF(mwi_msg, ast_smdi_mwi_message_destroy);
-                       ast_log(LOG_NOTICE, "Purged expired message from %s SMDI MWI message queue.  Message was %ld milliseconds too old.",
+                       ast_log(LOG_NOTICE, "Purged expired message from %s SMDI MWI message queue.  Message was %ld milliseconds too old.\n",
                                iface->name, elapsed - iface->msg_expiry);
                        mwi_msg = ASTOBJ_CONTAINER_UNLINK_START(&iface->mwi_q);
                }