Fixed problems with unit test tripping over undefined behavior.
authorJon A. Cruz <jonc@osg.samsung.com>
Thu, 23 Apr 2015 01:24:23 +0000 (18:24 -0700)
committerErich Keane <erich.keane@intel.com>
Thu, 23 Apr 2015 16:29:00 +0000 (16:29 +0000)
commit7b156d4f557a95088c6905c047183d1cfad7d738
tree27331495fd18ca38d6a050325434a4c524b0df51
parent8ac36258daea6c9bb539fa1ef2ba34287e3243b1
Fixed problems with unit test tripping over undefined behavior.

Corrected several test cases that caused crashes under glib due
to triggering undefined behavior in locked mutex clearing. Similar
prohibitions exist for pthreads also.

Added if() checks after EXPECT_TRUE cases that checked for NULL.
Without this the code will dereference null pointers when one
of the possible failure scenarios is encountered.

Change-Id: I232b52a715e7d3c0f7952756a7e98e6522b64dae
Signed-off-by: Jon A. Cruz <jonc@osg.samsung.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/813
Tested-by: jenkins-iotivity <jenkins-iotivity@opendaylight.org>
Reviewed-by: Joseph Morrow <joseph.l.morrow@intel.com>
Reviewed-by: Erich Keane <erich.keane@intel.com>
resource/csdk/connectivity/test/camutex_tests.cpp