- 24 May, 2017 4 commits
-
-
ol.beketov authored
Change-Id: I4cf4a6a01c64e8e455e51a3fca88e512b516e60c Signed-off-by:
Andrii Shtompel <a.shtompel@samsung.com> Signed-off-by:
ol.beketov <ol.beketov@samsung.com> Reviewed-on: https://gerrit.iotivity.org/gerrit/20325Tested-by:
jenkins-iotivity <jenkins@iotivity.org> Reviewed-by:
Uze Choi <uzchoi@samsung.com>
-
Dan Mihai authored
The local variable interfaces gets decremented during the loop, but the entire iflist has to be parsed. Fixes new test errors that started on my machine with recent change e3a3f655 : [ RUN ] CATests.GetNetworkInformationTest resource\csdk\connectivity\test\ca_api_unittest.cpp(517): error: Value of: tempInfo[index].adapter != 0 Actual: false Expected: true resource\csdk\connectivity\test\ca_api_unittest.cpp(518): error: Value of: strlen(tempInfo[index].addr) != 0 Actual: false Expected: true resource\csdk\connectivity\test\ca_api_unittest.cpp(517): error: Value of: tempInfo[index].adapter != 0 Actual: false Expected: true resource\csdk\connectivity\test\ca_api_unittest.cpp(518): error: Value of: strlen(tempInfo[index].addr) != 0 Actual: false Expected: true resource\csdk\connectivity\test\ca_api_unittest.cpp(517): error: Value of: tempInfo[index].adapter != 0 Actual: false Expected: true resource\csdk\connectivity\test\ca_api_unittest.cpp(518): error: Value of: strlen(tempInfo[index].addr) != 0 Actual: false Expected: true resource\csdk\connectivity\test\ca_api_unittest.cpp(517): error: Value of: tempInfo[index].adapter != 0 Actual: false Expected: true resource\csdk\connectivity\test\ca_api_unittest.cpp(518): error: Value of: strlen(tempInfo[index].addr) != 0 Actual: false Expected: true [ FAILED ] CATests.GetNetworkInformationTest (5 ms) Change-Id: If3a74a0739be6a53373f8b25436a50304029d751 Signed-off-by:
Dan Mihai <Daniel.Mihai@microsoft.com> Reviewed-on: https://gerrit.iotivity.org/gerrit/20337Reviewed-by:
Jaehong Jo <jaehong.jo@samsung.com> Tested-by:
-
arya.k authored
Change-Id: Id8c6c86421f156cad612417c70aa58064b3ab5ad Signed-off-by:
arya.k <arya.kumar@samsung.com> Reviewed-on: https://gerrit.iotivity.org/gerrit/20315Tested-by:
-
Sushil Yadav authored
In InProcServerWrapper::sendReponse() pResponse tries to dereference before checking for the null. The bug was introduced for as part of IOT-1583. Change-Id: Ic4ec8a08d3934d698af493703de1966b78702dba Signed-off-by:
Sushil Yadav <sushil.ky@samsung.com> Reviewed-on: https://gerrit.iotivity.org/gerrit/20319Reviewed-by:
-
- 23 May, 2017 10 commits
-
-
Greg Zaverucha authored
When deleteting roles with OCDeleteRoleCertificateByCredId, the callback provided by the caller was not being called. Change-Id: I51da5fe61760ff5561fd676e5c220905f106f512 Signed-off-by:
Greg Zaverucha <gregz@microsoft.com> Reviewed-on: https://gerrit.iotivity.org/gerrit/20269Tested-by:
Phil Coval <philippe.coval@osg.samsung.com> Reviewed-by:
Randeep Singh <randeep.s@samsung.com>
-
ol.beketov authored
Change-Id: I56923e53e9b81da45b9ff9e50e7df9cdde233c25 Signed-off-by:
-
Jaewook Jung authored
Add null checking for a resource object which could be NULL if server receives a request message including resource uri which server doesn't have. Change-Id: Idf29a43aeca5a919c0b22d3f0cd9ce507c8935ab Signed-off-by:
Jaewook Jung <jw0213.jung@samsung.com> Reviewed-on: https://gerrit.iotivity.org/gerrit/20297Reviewed-by:
-
Todd Malsbary authored
gtest.h: In instantiation of 'testing::AssertionResult \ testing::internal::CmpHelperNE(const char*, const char*, const \ T1&, const T2&) [with T1 = long int; T2 = char*]': rddatabase.cpp:126:9: required from here gtest.h:1577:28: error: ISO C++ forbids comparison between pointer \ and integer [-fpermissive] Change-Id: I6e5a2863fc6d45dffb5bc61526ae9f786eb1c638 Signed-off-by:Todd Malsbary <todd.malsbary@intel.com> Reviewed-on: https://gerrit.iotivity.org/gerrit/20263Reviewed-by:
-
Veeraj Khokale authored
1. Update dat files for proxy samples to be same as linux secure samples. 2. Modify logic to form server address according to linux secure samples while initiating proxy request. 3. Since request URI is NULL for proxy request check this condition before asserting role certificates in OCDoRequest(). This also removes the need for NULL check of request info resource uri. Change-Id: I20003ac7c7781666bbd4aa8955c4cc18b5a3262f Signed-off-by:
Veeraj Khokale <veeraj.sk@samsung.com> Signed-off-by:
Jihun Ha <jihun.ha@samsung.com> Reviewed-on: https://gerrit.iotivity.org/gerrit/20257Reviewed-by:
-
Jongmin Choi authored
IOT-2303. Comment modified to show optional parameter Change-Id: Iae07f2496ca3295d569ab0653afc8b61b0643605 Signed-off-by:
Jongmin Choi <jminl.choi@samsung.com> Reviewed-on: https://gerrit.iotivity.org/gerrit/20101Reviewed-by:
Md. Imtiaz Hossain <imtiaz.h@samsung.com> Tested-by:
dongik Lee <dongik.lee@samsung.com> Reviewed-by:
-
Todd Malsbary authored
dimensions[0] and i may be different values. The urlInfo is allocated to dimensions[0] but checked against i. Bug: https://jira.iotivity.org/browse/IOT-2344 Change-Id: Ibff3fc62d551417a71e7496027452b3d88cad413 Signed-off-by:
Srikrishna Gurugubelli <srikguru@microsoft.com> Tested-by:
Ashok Babu Channa <ashok.channa@samsung.com> Reviewed-by:
-
KIM JungYong authored
When RE build as Tizen, one of defination was redefined. In this patch, defination tag is changed. Change-Id: Ice25e9907481f66dac4b8a1e0aeb535120c936f0 Signed-off-by:
KIM JungYong <jyong2.kim@samsung.com> Reviewed-on: https://gerrit.iotivity.org/gerrit/20227Tested-by:
-
hyuna0213.jo authored
https://gerrit.iotivity.org/gerrit/#/c/19355/ was modified to make changes for all platforms. Change-Id: Ib653daaad453556354b108c673b34c06a197c5f9 Signed-off-by:
hyuna0213.jo <hyuna0213.jo@samsung.com> Reviewed-on: https://gerrit.iotivity.org/gerrit/19821Tested-by:
-
Jihun Ha authored
* Change minSdkVersion to 21 in AndroidManifest.xml * Change target source files for jni build ONLY to cpp files Change-Id: Ic45acb59d66d30c991514a2bec38583c0e7f1550 Signed-off-by:
Senthil Kumar G S <senthil.gs@samsung.com> Tested-by:
-
- 22 May, 2017 22 commits
-
-
Todd Malsbary authored
The checks added in commit 6758c572 made 'pri' mandatory again. Bug: https://jira.iotivity.org/browse/IOT-2175 Change-Id: Idb6137f45bf06910a9789f23c87486df2289abca Signed-off-by:
-
Oleksandr Dmytrenko authored
Update test due to DOS enabling Change-Id: I9f0c4b0b01ee3fe79a2126cd580cca4e6d374afd Signed-off-by:
Oleksandr Dmytrenko <o.dmytrenko@samsung.com> Reviewed-on: https://gerrit.iotivity.org/gerrit/20247Reviewed-by:
Dmitriy Zhuravlev <d.zhuravlev@samsung.com> Tested-by:
-
hyuna0213.jo authored
Remove assert role in case of TCP KeepAlive Patch #1: initial upload Bug: https://jira.iotivity.org/browse/IOT-2302 Change-Id: If99610a3a302357aa33ac0d1466219e78fdd01fa Signed-off-by:
-
jihwan.seo authored
Change-Id: I79f62fdd362542f1dc05cb77f986e34587c422e0 Signed-off-by:
jihwan.seo <jihwan.seo@samsung.com> Reviewed-on: https://gerrit.iotivity.org/gerrit/20223Reviewed-by:
-
Jihun Ha authored
If a response payload of discovery is NULL, which means discovery request is failed for some reason, it should not proceed a further operation. Only if a result of discovery request is OK, GET/PUT/POST requests can be performed, further. Change-Id: I95e79038dfae5e455f61db89aa9c7f3a724cee29 Signed-off-by:
Glen Youngjin Kim <glen.kim@samsung.com> Reviewed-by:
-
Srikrishna Gurugubelli authored
Change-Id: I0ae46770b08fb6b8c2d632189a060dc14e0e94c0 Signed-off-by:
Alex Kelley <alexke@microsoft.com> Reviewed-by:
Kevin Kane <kkane@microsoft.com> Tested-by:
-
Dan Mihai authored
Change-Id: Ib5e6df4e02da583e086a07591ef1e35f367ae4f0 Signed-off-by:
-
Way Vadhanasin authored
This change adds ability to create oc_mutex with recursive support. The function oc_mutex_new_recursive() will be used by a later change in OCStack to remove the responsibility of calling OCProcess from applications so that they don't have to call it in an endless loop. Change-Id: I0d60188af9bce866f5ba18800292204c78daba11 Signed-off-by:
Way Vadhanasin <wayvad@microsoft.com> Signed-off-by:
-
Way Vadhanasin authored
When a mutex goes through the condition variable wait API, there is a bug that causes oc_mutex_assert_owner to fail. This patch fixes that bug. Change-Id: I0d60188af9bce866f5ba18800292204c78daba12 Signed-off-by:
-
hyuna0213.jo authored
use caqueueingthread to send tls message in CATCPPacketSendCB Bug: https://jira.iotivity.org/browse/IOT-2284 Change-Id: Ie32ef9d9c593c3668232263ce9a58ad476986f83 Signed-off-by:
-
Jee Hyeok Kim authored
1. Revert cloud behavior not to disconnect client after cloud sends sign-up response to client. Change-Id: I367dfe4ebe16567ada67ae2f7aac8b8a4c0f5db2 Signed-off-by:
Jee Hyeok Kim <jihyeok13.kim@samsung.com> Reviewed-on: https://gerrit.iotivity.org/gerrit/20225Reviewed-by:
-
Ondrej Tomcik authored
Change-Id: I4ac25b7f605e355fc3aad286baa206d67584be69 Signed-off-by:
Ondrej Tomcik <ondrej.tomcik@kistler.com> Issue: IOT-2316 Reviewed-on: https://gerrit.iotivity.org/gerrit/20009Tested-by:
-
Jee Hyeok Kim authored
Change-Id: I10a04fcc499fa8de7fccfd6333d74f6688bb691b Signed-off-by:
-
Herve Jourdain authored
In the android case, this results in some jni libraries not being able to load the shared libraries they depend upon, which results in errors. The solution is to add the soname for the libraries that don't have it. Change-Id: I52e3e3e182b1aeb9257862c7253d11023a634ac3 Signed-off-by:
Herve Jourdain <herve.jourdain@beechwoods.com> Reviewed-on: https://gerrit.iotivity.org/gerrit/20105Reviewed-by:
-
Soemin Tjong authored
https://jira.iotivity.org/browse/IOT-1842 Removed the not working shallow copy of the OCRepresentation object. No problem is observed. Deep copy should also fix the problem if any issue is found in the future. Change-Id: Ifc4aba57a591d0cb6a55e65e1a179e2c8a9ebc08 Signed-off-by:
Soemin Tjong <stjong@microsoft.com> Reviewed-on: https://gerrit.iotivity.org/gerrit/19553Reviewed-by:
Dave Thaler <dthaler@microsoft.com> Tested-by:
-
Nathan Heldt-Sheller authored
The GetDefaultACL() function needed to be updated to /acl2 format. Fixed the printACL() function to support ace2 as well. Fixed aclresourcetest.cpp unit tests. Updated related .json and .dat files. Change-Id: Id1a14067e3c9d743cab489c733486fcea1e91d52 Signed-off-by:
Nathan Heldt-Sheller <nathan.heldt-sheller@intel.com> Reviewed-on: https://gerrit.iotivity.org/gerrit/19971Tested-by:
-
Jaewook Jung authored
- fixed memory leak - removed unreachable codes - add a null checking before refer a character pointer Change-Id: Ie49c44f0d2e5211dec24a607382c202900b4f6a3 Signed-off-by:
-
hyuna0213.jo authored
we should call CADestroyToken() to free memory after merging https://gerrit.iotivity.org/gerrit/19263 Bug: https://jira.iotivity.org/browse/IOT-2334 Bug: https://jira.iotivity.org/browse/IOT-2335 Change-Id: I10fb78d1f6f63a31678b07d917da547e054161b0 Signed-off-by:
-
KIM JungYong authored
1. unused parameters is resolved. 2. implicit using of functions is fixed as including owned header. 3. print format is fixed. Change-Id: I240ac626f6ed9f1aa02db7980ba657ae666653e9 Signed-off-by:
-
Philippe Coval authored
Problem was observed while building with: scons VERBOSE=1 RELEASE=true SECURED=0 WITH_MQ=PUB,SUB,BROKER service/notification/examples/linux/notificationconsumer.c:268:23: \ error: 'CLOUD_ADDRESS' undeclared (first use in this function) service/notification/examples/linux/notificationconsumer.c:268:23: \ note: each undeclared identifier is reported only once for each function it appears in Bug: https://jira.iotivity.org/browse/IOT-2244 Change-Id: I12f9a457ac7f838b90567794cbb030bdf80c3dc9 Signed-off-by:
-
Nathan Heldt-Sheller authored
Until AMS and /amacl functionality is implemented, the /amacl Resource should not be instantiated as it misleads Clients (including CTT) into expecting full Specification behavior. Change-Id: Id542b7a5974cb96ae84e7165326e4d2e8170d5dd Signed-off-by:
-
Mats Wichmann authored
service/easy-setup/enrollee/src/easysetup.c:182:9: warning: implicit declaration of function 'RegisterConnectRequestEventCallBack' [-Wimplicit-function-declaration] service/notification/examples/linux/notificationprovider.c:99:14: warning: implicit declaration of function 'strcmp' [-Wimplicit-function-declaration] Change-Id: I0e449b2834da90422f44e116c6ee91f241d6ec9e Signed-off-by:
Mats Wichmann <mats@linux.com> Reviewed-on: https://gerrit.iotivity.org/gerrit/18613Reviewed-by:
-
- 21 May, 2017 4 commits
-
-
Dan Mihai authored
Serialize the tests, because otherwise investigation of failures is too difficult. For an example of puzzling failure, see: https://build.iotivity.org/ci/job/iotivity-verify-windows-x86-vs2015/1495/ Change-Id: I90b836b4c4c6587de108de32d761f99fb3073819 Signed-off-by:
-
Nathan Heldt-Sheller authored
This workaround fixes issue reported in 2280, but the root cause of 2280 is that the /doxm POST handler still does all sorts of strange things per IOT-2107. IOT-2107 will certainly result in another failed TC shortly. I am changing IOT-2107 to P1 and to Fix In 1.3.0. This patch doesn't hurt in the meantime to resolve 2280. Change-Id: I2204cddabe943941f46136b60d0922a77f208665 Signed-off-by:
-
Nathan Heldt-Sheller authored
/acl2 POST handler now calls a version-check-only optional CBORPayloadToAcl() function and, if an acl1 payload is found, denies the POST request. This update caught some unit test issues that were also corrected with this patch. Change-Id: I20d148ef037c82f5862fd9fec156bbb399ab7417 Signed-off-by:
-
Dmitriy Zhuravlev authored
Skip /introspection resource in secure stack sample Change-Id: I36dd6718ba57d0712b22a147e3a7d2c638fe1c92 Signed-off-by:
-
