diff options
| author | rizzo <rizzo@f38db490-d61c-443f-a65b-d21fe96a405b> | 2007-11-14 13:18:40 +0000 |
|---|---|---|
| committer | rizzo <rizzo@f38db490-d61c-443f-a65b-d21fe96a405b> | 2007-11-14 13:18:40 +0000 |
| commit | ea0d4674a604b98a32c5f5b42f1878d3d75af9e0 (patch) | |
| tree | 01f0ca16e50b44357fb29bdb18793405a071da0f /main/logger.c | |
| parent | 2e70fd87205ec4f9c775075b807c65e3703145e3 (diff) | |
make the 'name' and 'value' fields in ast_variable const char *
This prevents modifying the strings in the stored variables,
and catched a few instances where this was actually done.
Given the differences between trunk and 1.4 (and the fact that this
is effectively an API change) it is better to fix 1.4 independently.
These are
chan_sip.c::sip_register()
chan_skinny.c:: near line 2847
config.c:: near line 1774
logger.c::make_components()
res_adsi.c:: near line 1049
I may have missed some instances for modules that do not build here.
git-svn-id: http://svn.digium.com/svn/asterisk/trunk@89268 f38db490-d61c-443f-a65b-d21fe96a405b
Diffstat (limited to 'main/logger.c')
| -rw-r--r-- | main/logger.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/main/logger.c b/main/logger.c index c1c8751e4..a5c7dd93a 100644 --- a/main/logger.c +++ b/main/logger.c @@ -174,11 +174,11 @@ AST_THREADSTORAGE(verbose_buf); AST_THREADSTORAGE(log_buf); #define LOG_BUF_INIT_SIZE 256 -static int make_components(char *s, int lineno) +static int make_components(const char *s, int lineno) { char *w; int res = 0; - char *stringp = s; + char *stringp = ast_strdupa(s); while ((w = strsep(&stringp, ","))) { w = ast_skip_blanks(w); @@ -204,7 +204,7 @@ static int make_components(char *s, int lineno) return res; } -static struct logchannel *make_logchannel(char *channel, char *components, int lineno) +static struct logchannel *make_logchannel(const char *channel, const char *components, int lineno) { struct logchannel *chan; char *facility; |