GitLab

fixed the review comments and applying the mbedtls patches
added stubbed function for st_wifi_free_scan_list in TizenRT
Added the latest commit changes on TizenRT and
auto build script for tizenrt.

Change-Id: I8fd4c901311c0bbd76854185aa70f558fb069bf3
Signed-off-by: pilli manikanta <p.manikanta@partner.samsung.com>
Signed-off-by: Harish Kumara M <h.marappa@samsung.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/26729Reviewed-by: Abhishek Pandey <abhi.siso@samsung.com>
Tested-by: IoTivity Jenkins <jenkins-daemon@iotivity.org>
Reviewed-by: Uze Choi <uzchoi@samsung.com>

Abort message of mbedtls optimization patch was updated
since before removed mbedtls implementation can't be detected
by abort print output.

Change-Id: I0133474a3a0f2b57c853c8762b6a01d0ce912e04
Signed-off-by: Aleksey Volkov <a.volkov@samsung.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/26759Tested-by: IoTivity Jenkins <jenkins-daemon@iotivity.org>
Reviewed-by: Jongmin Choi <jminl.choi@samsung.com>
Reviewed-by: Uze Choi <uzchoi@samsung.com>

Current:
  Only OTM logic is handling about fail case. So if provisioning(acl..)
  or no request for easy-setup routine is failed, then it can not go
  back to initialized status.
Solve:
  So Timeout check routine is added between after OTM success and easy-
  setup finish.

Additionally, some logs added a missed newline.

Change-Id: I01dfac7b87cf1f2e92062123c9fc25878c930257
Signed-off-by: Jaehyun Cho <jaehyun3.cho@samsung.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/26747Tested-by: IoTivity Jenkins <jenkins-daemon@iotivity.org>
Reviewed-by: Uze Choi <uzchoi@samsung.com>

Add using of EXPECT_NO_THROW macro
for all void function calls in
the security unittests

Change-Id: Ic3ed878b8f09fb834344f0aa6193f92dc2e4327f
Signed-off-by: Aleksey Volkov <a.volkov@samsung.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/26737Tested-by: IoTivity Jenkins <jenkins-daemon@iotivity.org>
Reviewed-by: Jongmin Choi <jminl.choi@samsung.com>
Reviewed-by: Uze Choi <uzchoi@samsung.com>

Added api comment in TC file
-base64test.cpp,colltest.cpp,ocapitest.cpp,reptest.cpp,uuidtest.cpp
-coaptest.cpp,enginetest.cpp,observetest.cpp,separatetest.cpp, transactiontest.cpp

Change-Id: I675e3849df98218963300a0b9b923b48480d75c2
Signed-off-by: Towhidul islam <t.islam@samsung.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/26741Tested-by: IoTivity Jenkins <jenkins-daemon@iotivity.org>
Reviewed-by: Uze Choi <uzchoi@samsung.com>

1. st_notify_back sometime error occur because of oc_main_init()
   is not called before.(oc_random_value() error)
2. cloud_manager_normal_test can be failed, when TC run several
   times. Because previous TC type is set into test_case_type.

Change-Id: I62355aca9bde9f4399d847e7d17f74bf847f1602
Signed-off-by: Jaehyun Cho <jaehyun3.cho@samsung.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/26749Tested-by: IoTivity Jenkins <jenkins-daemon@iotivity.org>
Reviewed-by: Uze Choi <uzchoi@samsung.com>

closing opened stream using "pclose" and
fixed a memory leak in easy-setup sample.

Change-Id: Id7dd8ce5271758fc43da40cc3ca5436e092fa8a9
Signed-off-by: Harish Kumara M <h.marappa@samsung.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/26751Tested-by: IoTivity Jenkins <jenkins-daemon@iotivity.org>
Reviewed-by: Abhishek Pandey <abhi.siso@samsung.com>
Reviewed-by: Uze Choi <uzchoi@samsung.com>

Fix a typo from 'prperties' to as 'properties'.
FIx incompatible data type when initializing type
st_property_t * using type st_property_t.

Change-Id: I5bf29c29b69a7861ef843da0da526b4c8832a036
Signed-off-by: uvasistha-samsung <utkarsh.m@samsung.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/26753Tested-by: IoTivity Jenkins <jenkins-daemon@iotivity.org>
Reviewed-by: Harish Marappa <h.marappa@samsung.com>
Reviewed-by: Abhishek Pandey <abhi.siso@samsung.com>
Reviewed-by: Uze Choi <uzchoi@samsung.com>

st_ref_app : string specifier does not have width limitation

Change-Id: I7843b9880713f22ab542bd7562128725931034fe
Signed-off-by: agrkush <kush.agrawal@samsung.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/26731Tested-by: IoTivity Jenkins <jenkins-daemon@iotivity.org>
Reviewed-by: Harish Marappa <h.marappa@samsung.com>
Reviewed-by: Uze Choi <uzchoi@samsung.com>

Given that IPV4 uses less memory, order of calling oc_dns_lookup is changed

Change-Id: I760fa3417c92706fa849d1dedbe9340711c715dc
Signed-off-by: iot <rami.jung@samsung.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/26479Tested-by: IoTivity Jenkins <jenkins-daemon@iotivity.org>
Reviewed-by: Jaehong Jo <jaehong.jo@samsung.com>
Reviewed-by: jaehyun Cho <jaehyun3.cho@samsung.com>
Reviewed-by: Uze Choi <uzchoi@samsung.com>

Check whether device index is equal or not.

Change-Id: I675e3849df98218963300a0b9b923b48480d75c1
Signed-off-by: Jaehyun Cho <jaehyun3.cho@samsung.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/26715Tested-by: IoTivity Jenkins <jenkins-daemon@iotivity.org>
Reviewed-by: Jaehong Jo <jaehong.jo@samsung.com>
Reviewed-by: Uze Choi <uzchoi@samsung.com>

discovery wait time logic is not working proper.

Change-Id: If9ae870c71f8821239b9cd310486f1473fd51b71
Signed-off-by: Jaehyun Cho <jaehyun3.cho@samsung.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/26717Tested-by: IoTivity Jenkins <jenkins-daemon@iotivity.org>
Reviewed-by: Jaehong Jo <jaehong.jo@samsung.com>
Reviewed-by: Uze Choi <uzchoi@samsung.com>

Add  Assert No Thrown Exception in some test case below:
+ FreeServerEndpoint_P
+ FreeServerEndpoint_N
+ CloseSession_P

Change-Id: I3bf5e02cd70222f8037b327a024cd1a50b4aaa8a
Signed-off-by: Jaehong Jo <jaehong.jo@samsung.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/26727Tested-by: IoTivity Jenkins <jenkins-daemon@iotivity.org>
Reviewed-by: jaehyun Cho <jaehyun3.cho@samsung.com>
Reviewed-by: Uze Choi <uzchoi@samsung.com>

add  Assert No Thrown Exception in some test case below:
 + DeleteCollectionTest_P
 + DeleteLinkTest_P
 + AddLinkRelationTest_P
 + SetLinkInstanceTest_P
 + AddCollectionLinkTest_P
 + RemoveCollectionLinkTest_P

Change-Id: Ia81c830dc8862b8116f5f0e9f8f078e587921c16
Signed-off-by: Towhidul islam <t.islam@samsung.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/26725Tested-by: IoTivity Jenkins <jenkins-daemon@iotivity.org>
Reviewed-by: Jaehong Jo <jaehong.jo@samsung.com>
Reviewed-by: Uze Choi <uzchoi@samsung.com>

The following functiones are restored:
x509_get_ext_key_usage()
x509_memcasecmp()

Change-Id: I62d01ac79d85c377a6d763197e2f79d686a5391f
Signed-off-by: Oleksii Beketov <ol.beketov@samsung.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/26697Tested-by: IoTivity Jenkins <jenkins-daemon@iotivity.org>
Reviewed-by: Jongmin Choi <jminl.choi@samsung.com>
Reviewed-by: Uze Choi <uzchoi@samsung.com>

Some TC failed because only refer EXTRA_CFLAGS
So Moved OC_SPEC_VER_OIC to EXTRA_CFLAGS from CFLAGS

Change-Id: Ie5fad5ba36e4198a1cfee6577b5d90f60e0c0501
Signed-off-by: Jaehong Jo <jaehong.jo@samsung.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/26719Tested-by: IoTivity Jenkins <jenkins-daemon@iotivity.org>
Reviewed-by: jaehyun Cho <jaehyun3.cho@samsung.com>
Reviewed-by: Uze Choi <uzchoi@samsung.com>

Common in debug mode mbedtls printed all messages,
now by default will print only error.
The new compile flag (OC_)DEBUG_TLS turn on full debug.

Change-Id: I19b1e99a58b12a9ac34678359a8d5eed026a2d12
Signed-off-by: Oleksandr Dmytrenko <o.dmytrenko@samsung.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/26695Tested-by: IoTivity Jenkins <jenkins-daemon@iotivity.org>
Reviewed-by: Jongmin Choi <jminl.choi@samsung.com>
Reviewed-by: Uze Choi <uzchoi@samsung.com>

st_notify_back

Change-Id: Ia81c830dc8862b8116f5f0e9f8f078e587921c15
Signed-off-by: Hanna Cha <hanna.cha@samsung.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/26699Tested-by: IoTivity Jenkins <jenkins-daemon@iotivity.org>
Reviewed-by: Uze Choi <uzchoi@samsung.com>

Change-Id: I85cb59ba6f18433c53bd31a78d75e19ea0409792
Signed-off-by: iot <rami.jung@samsung.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/26685Tested-by: IoTivity Jenkins <jenkins-daemon@iotivity.org>
Reviewed-by: Uze Choi <uzchoi@samsung.com>

Change-Id: Id477e3808d2740a1363691915baf172b391930a7
Signed-off-by: Harish Kumara M <h.marappa@samsung.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/26677Tested-by: IoTivity Jenkins <jenkins-daemon@iotivity.org>
Reviewed-by: Uze Choi <uzchoi@samsung.com>

Enclose MFG specific routines

Change-Id: Id1ac6d9b0d7a98eb8d140c842246bb4837a3bb30
Signed-off-by: Oleksii Beketov <ol.beketov@samsung.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/26537Tested-by: IoTivity Jenkins <jenkins-daemon@iotivity.org>
Reviewed-by: Uze Choi <uzchoi@samsung.com>

Easysetup_linux - String not NULL terminated

Change-Id: I24832ccfb767c8af9b5d4bdc3b790862b9c9c1cf
Signed-off-by: agrkush <kush.agrawal@samsung.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/26683Tested-by: IoTivity Jenkins <jenkins-daemon@iotivity.org>
Reviewed-by: Uze Choi <uzchoi@samsung.com>

Replace ST_APP_OPTIMIZATION with ST_MBEDTLS_OPT
in mbedtls/config.h

Change-Id: I201ca8d2688ee24e3a1aafc2951795aafe28c6cd
Signed-off-by: Oleksii Beketov <ol.beketov@samsung.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/26635Tested-by: IoTivity Jenkins <jenkins-daemon@iotivity.org>
Reviewed-by: Jongmin Choi <jminl.choi@samsung.com>
Reviewed-by: Uze Choi <uzchoi@samsung.com>

Next round of mbedtls optimization (experimental)

Change-Id: Idcb1454d1db8a655bf3e174def5431d79bbd18a0
Signed-off-by: Oleksii Beketov <ol.beketov@samsung.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/26571Tested-by: IoTivity Jenkins <jenkins-daemon@iotivity.org>
Reviewed-by: Jongmin Choi <jminl.choi@samsung.com>
Reviewed-by: Uze Choi <uzchoi@samsung.com>

To use EXPECT_FALSE instead EXPECT_TRUE 
because expected result is not ES_OK when wrong case.

Change-Id: Ia18b7f7d1b538823728e7e7be9ed516414aea4a5
Signed-off-by: samanwayd <samanway.dey@samsung.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/26673Reviewed-by: Uze Choi <uzchoi@samsung.com>
Tested-by: Uze Choi <uzchoi@samsung.com>

Sleep is replaced by cond/signal in ocapitest

Change-Id: I31971a1002542faa53d5e774ceb2f1a7f69d9085
Signed-off-by: Mushfiqul Islam Antu <i.mushfiq@samsung.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/26471Tested-by: IoTivity Jenkins <jenkins-daemon@iotivity.org>
Reviewed-by: Uze Choi <uzchoi@samsung.com>

added oc_rep test case for all defined macro

Change-Id: Idfa19cf3acad94cc21a2e00a9dbc8856199681d2
Signed-off-by: Towhidul islam <t.islam@samsung.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/26629Reviewed-by: Uze Choi <uzchoi@samsung.com>
Tested-by: IoTivity Jenkins <jenkins-daemon@iotivity.org>

As such OCF spec does not mandate the nature
of array in resource representation. Stack is
implemented in the assumption of receiving
homogeneous array always, but this may not be
true in case of third part OCF servers/clients.
Changed the parsing logic to avoid crash in case
of handling heterogeneous array.

Change-Id: I0962cdb2c17988b1ff630c7c532cc20ef843998b
Signed-off-by: Harish Kumara M <h.marappa@samsung.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/26625Reviewed-by: Abhishek Pandey <abhi.siso@samsung.com>
Tested-by: IoTivity Jenkins <jenkins-daemon@iotivity.org>
Reviewed-by: Uze Choi <uzchoi@samsung.com>

1) cred: cert and mfgcert was dynamic allocated, in case error(return) memory leak.
2) security unittest: wrong response length fixed.
3) tls: pkey free, hmac function return checks

Change-Id: I2d320280ed5d387586db7b53cd3e7ed035192ef9
Signed-off-by: Oleksandr Dmytrenko <o.dmytrenko@samsung.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/26603Tested-by: IoTivity Jenkins <jenkins-daemon@iotivity.org>
Reviewed-by: Uze Choi <uzchoi@samsung.com>

added utc in reptest.cpp for oc_rep file

Change-Id: Idfa19cf3acad94cc21a2e00a9dbc8856199681d1
Signed-off-by: Towhidul islam <t.islam@samsung.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/26621Tested-by: IoTivity Jenkins <jenkins-daemon@iotivity.org>
Reviewed-by: Uze Choi <uzchoi@samsung.com>

Change-Id: Idfa19cf3acad94cc21a2e00a9dbc8856199681df
Signed-off-by: Harish Kumara M <h.marappa@samsung.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/26617Tested-by: IoTivity Jenkins <jenkins-daemon@iotivity.org>
Reviewed-by: Uze Choi <uzchoi@samsung.com>

Some OS is not work properly using signal. Until figure out
exact problem to prevent signaling, it will be disabled.

Change-Id: I61f230165458299e49788fd27f63ce7c9b79285b
Signed-off-by: Jaehyun Cho <jaehyun3.cho@samsung.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/26607Tested-by: IoTivity Jenkins <jenkins-daemon@iotivity.org>
Reviewed-by: Uze Choi <uzchoi@samsung.com>

RPK API, error callback, device's OTM method setter
and OTM user callback gathered into security.h

Change-Id: I0f138af190ad1d685ed371869f40348d6846952e
Signed-off-by: Oleksii Beketov <ol.beketov@samsung.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/26429Tested-by: IoTivity Jenkins <jenkins-daemon@iotivity.org>
Reviewed-by: Uze Choi <uzchoi@samsung.com>

In some environments, signal handling we not working properly.
So st_cond_wait is changed to st_cont_timedwait.

Change-Id: Iffc845eca3f643640f75ee322147a67f1a642c63
Signed-off-by: Jaehyun Cho <jaehyun3.cho@samsung.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/26599Tested-by: IoTivity Jenkins <jenkins-daemon@iotivity.org>
Reviewed-by: Uze Choi <uzchoi@samsung.com>

Formatting changes in sc_easy_setup.h

Change-Id: Ic359587c226b29d5105bea2b377dc87934b136f2
Signed-off-by: agrkush <kush.agrawal@samsung.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/26597Tested-by: IoTivity Jenkins <jenkins-daemon@iotivity.org>
Reviewed-by: Uze Choi <uzchoi@samsung.com>

Fix typo in mbedtls-reduce.patch

Change-Id: I0e1e4e7dc93f7ea78df025e7552af51387f2585b
Signed-off-by: Oleksii Beketov <ol.beketov@samsung.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/26567Tested-by: IoTivity Jenkins <jenkins-daemon@iotivity.org>
Reviewed-by: Jongmin Choi <jminl.choi@samsung.com>
Reviewed-by: Uze Choi <uzchoi@samsung.com>

AS-IS : failed when retry count over
TO-BE : close session and restart connection

Change-Id: I3f990cec964afe0312fb8cece6042908ea5a488c
Signed-off-by: Jaehong Jo <jaehong.jo@samsung.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/26485Tested-by: IoTivity Jenkins <jenkins-daemon@iotivity.org>
Reviewed-by: Uze Choi <uzchoi@samsung.com>

To set proper configuration info from json file, st_data_manager
have st_configuration_t struct value which is parsed into json file.
In st_manager will set into iotivity-lite stack refer from this.
Regarding RPK handling logic, make register/unregister api to handle
RPK key setting logic.

Change-Id: I4e789bf17da014996d4f8ac90063fda8dd33886a
Signed-off-by: Jaehyun <jaehyun3.cho@samsung.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/26491Tested-by: IoTivity Jenkins <jenkins-daemon@iotivity.org>
Reviewed-by: Jaehong Jo <jaehong.jo@samsung.com>
Reviewed-by: Uze Choi <uzchoi@samsung.com>

In case of deleting resource using oc_delete_resource(),
add remove observer logic in delete resource logic.
If observer is not removed when delete resource, it will
occur crash when remove every observers from shutdown(because
observer has resource pointer in their member struct).

Change-Id: Ic71b90b76fce43df2ca7d8aa2a3904d892959044
Signed-off-by: Jaehyun Cho <jaehyun3.cho@samsung.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/26577Tested-by: IoTivity Jenkins <jenkins-daemon@iotivity.org>
Reviewed-by: Uze Choi <uzchoi@samsung.com>

Change-Id: I32f7e2649355de7e0da140dc351244f9b9da3b25
Signed-off-by: Jaehong Jo <jaehong.jo@samsung.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/26553Tested-by: IoTivity Jenkins <jenkins-daemon@iotivity.org>
Reviewed-by: Uze Choi <uzchoi@samsung.com>