Merged revisions 209759 via svnmerge from

https://origsvn.digium.com/svn/asterisk/branches/1.4

........
  r209759 | kpfleming | 2009-07-31 19:52:00 -0500 (Fri, 31 Jul 2009) | 7 lines
  
  Minor changes inspired by testing with latest GCC.
  
  The latest GCC (what will become 4.5.x) has a few new warnings, that in these
  cases found some either downright buggy code, or at least seriously poorly
  designed code that could be improved.
........


git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@209760 65c4cc65-6c06-0410-ace0-fbb531ad65f3
certified/1.8.6
Kevin P. Fleming 16 years ago
parent 68a05b14c6
commit ed2a3cedd1

@ -237,7 +237,7 @@ ifeq ($(AST_DEVMODE),yes)
_ASTCFLAGS+=$(AST_FORTIFY_SOURCE) _ASTCFLAGS+=$(AST_FORTIFY_SOURCE)
_ASTCFLAGS+=-Wundef _ASTCFLAGS+=-Wundef
_ASTCFLAGS+=-Wmissing-format-attribute _ASTCFLAGS+=-Wmissing-format-attribute
_ASTCFLAGS+=-Wformat=2 _ASTCFLAGS+=-Wformat=2 -Wno-format-security
endif endif
ifneq ($(findstring BSD,$(OSARCH)),) ifneq ($(findstring BSD,$(OSARCH)),)

@ -2526,7 +2526,7 @@ static void my_pri_fixup_chans(void *old_chan, void *new_chan)
old->dsp_features = 0; old->dsp_features = 0;
} }
static int sig_pri_tone_to_dahditone(enum analog_tone tone) static int sig_pri_tone_to_dahditone(enum sig_pri_tone tone)
{ {
switch (tone) { switch (tone) {
case SIG_PRI_TONE_RINGTONE: case SIG_PRI_TONE_RINGTONE:

@ -11081,7 +11081,7 @@ static int misdn_command_cc_deactivate(struct ast_channel *chan, struct misdn_co
error_str = misdn_no_response_from_network; error_str = misdn_no_response_from_network;
} else if (cc_record->reject_code != FacReject_None) { } else if (cc_record->reject_code != FacReject_None) {
error_str = misdn_to_str_reject_code(cc_record->reject_code); error_str = misdn_to_str_reject_code(cc_record->reject_code);
} else if (cc_record->reject_code != FacError_None) { } else if (cc_record->error_code != FacError_None) {
error_str = misdn_to_str_error_code(cc_record->error_code); error_str = misdn_to_str_error_code(cc_record->error_code);
} else { } else {
error_str = NULL; error_str = NULL;

@ -247,7 +247,7 @@ static void dec_ie_bearer(unsigned char *p, Q931_info_t *qi, int *coding, int *c
if (p[0] <= octet) if (p[0] <= octet)
goto done; goto done;
if (!p[octet++] & 0x80) if (~p[octet++] & 0x80)
goto l2; goto l2;
/* Wheee. V.110 speed information */ /* Wheee. V.110 speed information */

@ -98,7 +98,7 @@ editline/libedit.a: CHECK_SUBDIR
$(MAKE) -C editline libedit.a $(MAKE) -C editline libedit.a
db1-ast/libdb1.a: CHECK_SUBDIR db1-ast/libdb1.a: CHECK_SUBDIR
CFLAGS="$(subst $(ASTTOPDIR),../../,$(_ASTCFLAGS) $(ASTCFLAGS))" LDFLAGS="$(ASTLDFLAGS)" $(MAKE) -C db1-ast libdb1.a _ASTCFLAGS="$(_ASTCFLAGS)" ASTCFLAGS="$(ASTCFLAGS) -Wno-strict-aliasing" $(MAKE) -C db1-ast libdb1.a
ast_expr2.c ast_expr2.h: ast_expr2.c ast_expr2.h:
bison -o $@ -d --name-prefix=ast_yy ast_expr2.y bison -o $@ -d --name-prefix=ast_yy ast_expr2.y

@ -1440,7 +1440,7 @@ static int pbx_load_config(const char *config_file)
goto process_extension; goto process_extension;
} }
} else if (!strcasecmp(v->name, "exten")) { } else if (!strcasecmp(v->name, "exten")) {
int ipri = -2; int ipri;
char *plus, *firstp; char *plus, *firstp;
char *pri, *appl, *data, *cidmatch; char *pri, *appl, *data, *cidmatch;
@ -1452,6 +1452,7 @@ static int pbx_load_config(const char *config_file)
pbx_substitute_variables_helper(NULL, ext, realext, sizeof(realext) - 1); pbx_substitute_variables_helper(NULL, ext, realext, sizeof(realext) - 1);
ast_copy_string(lastextension, realext, sizeof(lastextension)); ast_copy_string(lastextension, realext, sizeof(lastextension));
process_extension: process_extension:
ipri = -2;
if ((cidmatch = strchr(realext, '/'))) { if ((cidmatch = strchr(realext, '/'))) {
*cidmatch++ = '\0'; *cidmatch++ = '\0';
ast_shrink_phone_number(cidmatch); ast_shrink_phone_number(cidmatch);

@ -744,7 +744,7 @@ void checknoargs( int argcount, char *args[])
void parseargs( int argcount, char *args[], int fileswitch) void parseargs( int argcount, char *args[], int fileswitch)
{ {
char *filename; char *filename;
int tempint; int tempint = 0;
if ((fileswitch & 1) != 0) /* If getting infile */ if ((fileswitch & 1) != 0) /* If getting infile */
in = NULL; in = NULL;

Loading…
Cancel
Save