summaryrefslogtreecommitdiffstats
path: root/unit/test-common.c
diff options
context:
space:
mode:
authorAndrzej Zaborowski <andrew.zaborowski@intel.com>2010-06-21 10:41:10 +0200
committerDenis Kenzior <denkenz@gmail.com>2010-06-21 15:21:40 -0500
commitd5fec53d23dbdece08ab94e2f02b910bffd9f766 (patch)
treeae100a6a19f9e018785bf7a3cb98645b7943ddff /unit/test-common.c
parentb8f68a5ed3aec89da5f36bbd1b00024ed7cb414c (diff)
downloadofono-d5fec53d23dbdece08ab94e2f02b910bffd9f766.tar.bz2
test-common: Avoid accessing an unitialised variable.
Fixes a segfault, reported by valrgind as: [...] ==6300== Use of uninitialised value of size 4 ==6300== at 0x8048B4D: test_invalid (test-common.c:81)
Diffstat (limited to 'unit/test-common.c')
-rw-r--r--unit/test-common.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/unit/test-common.c b/unit/test-common.c
index f33f117e..baed1785 100644
--- a/unit/test-common.c
+++ b/unit/test-common.c
@@ -78,7 +78,7 @@ static void test_invalid()
ret = parse_ss_control_string(str, &type, &sc,
&sia, &sib, &sic, &sid, &dn);
- if (strlen(sid))
+ if (ret == TRUE && strlen(sid))
ret = FALSE;
g_assert(ret == FALSE);