diff options
author | Matthew Jordan <mjordan@digium.com> | 2015-04-11 15:27:08 +0000 |
---|---|---|
committer | Matthew Jordan <mjordan@digium.com> | 2015-04-11 15:27:08 +0000 |
commit | 356b770632d34d6e64792bfc0427f4aec1676180 (patch) | |
tree | f3d61cd9ad426acbd72218ff11ba349dbc784e48 /tests/test_strings.c | |
parent | 5f181bcccd9f3309958f341c9fb9e9b0a307c2f9 (diff) |
clang compiler warnings: Fix various warnings for tests
This patch fixes a variety of clang compiler warnings for unit tests. This
includes autological comparison issues, ignored return values, and
interestingly enough, one embedded function. Fun!
Review: https://reviewboard.asterisk.org/r/4555
ASTERISK-24917
Reported by: dkdegroot
patches:
rb4555.patch submitted by dkdegroot (License 6600)
........
Merged revisions 434705 from http://svn.asterisk.org/svn/asterisk/branches/11
........
Merged revisions 434706 from http://svn.asterisk.org/svn/asterisk/branches/13
git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@434707 65c4cc65-6c06-0410-ace0-fbb531ad65f3
Diffstat (limited to 'tests/test_strings.c')
-rw-r--r-- | tests/test_strings.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/tests/test_strings.c b/tests/test_strings.c index cf089a865..4321d4a03 100644 --- a/tests/test_strings.c +++ b/tests/test_strings.c @@ -390,9 +390,12 @@ AST_TEST_DEFINE(strsep_test) static int test_semi(char *string1, char *string2, int test_len) { char *test2 = NULL; - if (test_len >= 0) { + + if (test_len > 0) { test2 = ast_alloca(test_len); *test2 = '\0'; + } else if (test_len == 0) { + test2 = ""; } ast_escape_semicolons(string1, test2, test_len); if (test2 != NULL && strcmp(string2, test2) == 0) { |