- 06 Dec, 2016 1 commit
-
-
Habib Virji authored
setPropertyValue call in the Android was swapped, compared to function call how it was in the JNI layer. setPropertyValue0 is for passing string array and setPropertyValue1 is passing string. Change-Id: I91c480f162cc53712a68bef078371559fd2eff21 Signed-off-by:
Habib Virji <habib.virji@samsung.com> Reviewed-on: https://gerrit.iotivity.org/gerrit/15025Tested-by:
jenkins-iotivity <jenkins-iotivity@opendaylight.org> Reviewed-by:
Phil Coval <philippe.coval@osg.samsung.com> Reviewed-by:
Uze Choi <uzchoi@samsung.com> (cherry picked from commit c54aaa1f) Reviewed-on: https://gerrit.iotivity.org/gerrit/15067Reviewed-by:
Larry Sachs <larry.j.sachs@intel.com> Reviewed-by:
Rick Bell <richard.s.bell@intel.com>
-
- 05 Dec, 2016 23 commits
-
-
Philippe Coval authored
Harmless change to test CI Change-Id: I3aa34dbdeb8905f53939d6eaee90a681bf85d730 Signed-off-by:
-
George Nash authoredAdd the symbols for mBedTLS if building a secure build Change-Id: Ibbe2387307bc432193daaa2b7f88491cb18687bd Signed-off-by:
George Nash <george.nash@intel.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/14687Tested-by: Kevin Kane <kkane@microsoft.com> Reviewed-by:
-
George Nash authored
This change cleans up the build output from android service samples - easy-setup - notification - resource-container - resource-encapsulation Change-Id: I942e365fd787387a30d6358f4586b9caa98fbeda Signed-off-by:
George Nash <george.nash@intel.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/14467Tested-by: -
George Nash authored
the std::string ret is a local varaible and is destroyed when the what() member function returns. Put the return string into the m_whatMessage variable. Since 'what()' member function is a const function it can not modify the member variable m_whatMessage so it is generated in the ResourceInitException constructor. Issue found using static analysis tool. Change-Id: I907b984f35dee59b2f300afe6a640b15a26f020f Signed-off-by:
George Nash <george.nash@intel.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/15035Tested-by: Dan Mihai <Daniel.Mihai@microsoft.com>
-
bg.chun authored
On android ipadapter there is memory leak issue since the function which used for free pointer of structure does not free all pointer member Change-Id: If44dd9dc7562f851fa3f38526830aa84e4b35491 Signed-off-by:
bg.chun <bg.chun@samsung.com> Reviewed-on: https://gerrit.iotivity.org/gerrit/14997Tested-by:
Jaehong Jo <jaehong.jo@samsung.com> Reviewed-by:
-
jaehyun Cho authored
Signed-off-by:
jaehyun Cho <jaehyun3.cho@samsung.com> Change-Id: Id98b2151b8149d892d1e063ca893c85fd672af50 Reviewed-on: https://gerrit.iotivity.org/gerrit/14353Reviewed-by:
jihwan seo <jihwan.seo@samsung.com> Reviewed-by:
Ashok Babu Channa <ashok.channa@samsung.com>
-
jihwan.seo authored
since some user application want to use IoTivity more dynamically. stop API of IoTivity should be provided effeciently. Change-Id: Ie9ebb874b9b1d49703d5023a972cff5c630acaf3 Signed-off-by:
-
Ziran Sun authored
Using ServiceType::OutofProc causes exception throw in registerResource unit test. Replace with ServiceType::InProc. Change-Id: I89ea8741c19af2567f9ae84dc88983a8e54fb64f Signed-off-by:
Ziran Sun <ziran.sun@samsung.com> Reviewed-on: https://gerrit.iotivity.org/gerrit/10159Tested-by:
-
saurabh.s9 authored
Patch #11 Fix Review comments Changed MACRO _ENABLE_MULTIPLE_OWNER_ to MULTIPLE_OWNER and this change propagated upto C-API's. Change-Id: I55305d5cf9210b25c68f0eaf04c1362111735979 Signed-off-by:
saurabh.s9 <saurabh.s9@samsung.com> Signed-off-by:
Sandeep Sharma <sandeep.s9@samsung.com> Signed-off-by:
Randeep Singh <randeep.s@samsung.com> Reviewed-on: https://gerrit.iotivity.org/gerrit/14221Tested-by:
Chul Lee <chuls.lee@samsung.com> Signed-off-by:
-
YounghyunJoo authored
- before this commit, gbsbuild script file can get a few build options like as "WITH_TCP, WITH_CLOUD, WITH_PROXY, etc.." - so, to add other option, need to modify gbsbuild scriptfile. - in this commit, I modified gbsbuild script file to support other options - using bash script, gets other option like way to get option in other platform - ex) ./gbsbuild.sh WITH_TCP=1 WITH_CLOUD=1 SECURED=1 WITH_MQ=PUB,SUB Change-Id: I61c0b02fe68acf517b64de7e3a8a10fe2549c3d2 Signed-off-by:
YounghyunJoo <yh_.joo@samsung.com> Reviewed-on: https://gerrit.iotivity.org/gerrit/14637Reviewed-by:
JungYong KIM <jyong2.kim@samsung.com> Tested-by:
JunHo Lee <junho13.lee@samsung.com> Reviewed-by:
daeken kwon <daeken.kwon@samsung.com> Reviewed-by:
-
sy01.youn authored
There was a build error when building enrollee sample app based on tizen, because of missing ocrandom.h file in iotivity.spec file. Change-Id: I3e20cac04d2691d58fc291ea88e2983845e9fbd6 Signed-off-by:
sy01.youn <sy01.youn@samsung.com> Reviewed-on: https://gerrit.iotivity.org/gerrit/15127Tested-by:
-
jihwan.seo authored
Change-Id: I7d88f7f140781ea23db02d0bfbf00f66cd145a13 Signed-off-by:
-
jihwan.seo authored
Change-Id: If324fc9399f48ce64d759d016da124963e136ea9 Signed-off-by:
-
Minji Park authored
- Changed error handling logic when tls message send failed in tcpadapter. Change-Id: I22d11d34d2092a7f452d41be9f243dcfdf16a02d Signed-off-by:
Hyuna Jo <hyuna0213.jo@samsung.com> Signed-off-by:
Minji Park <minjii.park@samsung.com> Reviewed-on: https://gerrit.iotivity.org/gerrit/14493Tested-by:
-
Minji Park authored
- Add shutdown() before close socket when closing connection in catcpserver Change-Id: Ibe2ed942428f754dc148bd1c48422036f84a319c Signed-off-by:
-
jihwan.seo authored
Change-Id: I5b0873a56e8c98094412b536309fa77f3643d126 Signed-off-by:
-
jihwan.seo authored
to make sure of analyze transmission state, all messages related important code point is printed same log method. - print CoAP message. - print send result. - print adapter state callback. Change-Id: Iccd8c8e9351165c7a6feb477e62b3b8814af0955 Signed-off-by:
-
KIM JungYong authored
In this patch, incorrect coding rule is fixed and resolved warning of static analysis. 1. Uninitialized variables is fixed with initialized. 2. Invalid bracket location is fixed. 3. Incorrect division with zero is prevented. 4. Incorrect referenced pointer is prevented. Change-Id: I715d671241f3d48a1174c49cbe34074b31a50f02 Signed-off-by:
-
jaesick.shin authored
When a consumer had control, they did not send a response So I modified it to respond. and remove duplicate defined about NS_QUERY_COSUMER_ID In the commit below, Committed without adding the file. https://gerrit.iotivity.org/gerrit/#/c/14657/ Change-Id: Ie38aec8895422d9edab9d46bef7c6b17499f5bce Signed-off-by:
jaesick.shin <jaesick.shin@samsung.com> Reviewed-on: https://gerrit.iotivity.org/gerrit/14709Reviewed-by:
Chihyun Cho <ch79.cho@samsung.com> Reviewed-by:
-
jaesick.shin authored
All the OCF property should be written as lower case. So, change the key name of attributes. second patch include, removed duplicated NS_QUERY_CONSUMERID. Change-Id: I2fcc76fac003e805ad19edb0d7ee10c659218b5f Signed-off-by:
-
jaesick.shin authored
This patch include, missing the declare of function. Change-Id: I1b543402a9ba8487029d8b59e806c6b570439652 Signed-off-by:
-
Ziran Sun authored
Change-Id: I57863e2291451aea5f1ce6d5fb7059b29d68a102 Signed-off-by:
-
Parkhi authored
Change-Id: I9eba31f8f716f8d3b5b04b803bc4c9ecdf035e10 Signed-off-by:
Parkhi <h_w.park@samsung.com> Reviewed-on: https://gerrit.iotivity.org/gerrit/15007Reviewed-by:
Jihun Ha <jihun.ha@samsung.com> Reviewed-by:
Madan Lanka <lanka.madan@samsung.com> Tested-by:
-
- 02 Dec, 2016 8 commits
-
-
srijon.sb authored
Change-Id: I3855ed16970eb5a0cb686ffe396410c6e0934709 Signed-off-by:
srijon.sb <srijon.sb@samsung.com> Reviewed-on: https://gerrit.iotivity.org/gerrit/15047Tested-by:
-
Dave Thaler authored
Fix memory corruption bugs in OCSetAttribute introduced by https://gerrit.iotivity.org/gerrit/#/c/14377/ which left pointers to freed memory in the attribute list, which was then later used. Change-Id: Ib776d4bd14aab7b39c74188246005628fbe18ecf Signed-off-by:
Dave Thaler <dthaler@microsoft.com> Reviewed-on: https://gerrit.iotivity.org/gerrit/14873Tested-by:
-
bg.chun authored
Modify tizen bt_edr apdapter to apply new logger which is Analyzer. Change-Id: I49d57a1d633c1d899494567080763f4ebdb26ccb Signed-off-by:
-
Jongmin Choi authored
If an error is returned during ownership transfer, on-boarding tool needs to remove the server's information from PDM DB and close secure connection with the server. Patch #1: Initial patch Patch #2: Change Log Level Change-Id: I641529552de0e25438bd94da9b2ec45320428a61 Signed-off-by:
Jongmin Choi <jminl.choi@samsung.com> Reviewed-on: https://gerrit.iotivity.org/gerrit/14999Reviewed-by:
Jongsung Lee <js126.lee@samsung.com> Reviewed-by:
-
Todd Malsbary authored
OCRepPayloads with values having only sequential integer names are encoded/decoded as CBOR arrays of mixed types. Change-Id: I2cdd87f9fb4b8473e00aefdd5bc5daa74941290f Bug: https://jira.iotivity.org/browse/IOT-1116Signed-off-by:
Todd Malsbary <todd.malsbary@intel.com> Reviewed-on: https://gerrit.iotivity.org/gerrit/13661Tested-by:
-
Parkhi authored
Change-Id: Id7d1deb6371d49e4810c191bc52979693ce16e8d Signed-off-by:
-
jihwan.seo authored
Change-Id: I7803348f17ac8b860863e25a35c6a12b73a5886d Signed-off-by:
-
jihwan.seo authored
Change-Id: Ieffef039491a8895c359efb807dc36daa55860d1 Signed-off-by:
-
- 01 Dec, 2016 3 commits
-
-
Jaewook Jung authored
If coap_add_option fails, it has to return fail while sending data. Otherwise, it causes a crash error when it tries to use a pdu wrong. The below is the comment for coap_add_option on libcoap-4.1.1 /** * Adds option of given type to pdu that is passed as first parameter. * coap_add_option() destroys the PDU's data, so coap_add_data() must be called * after all options have been added. As coap_add_token() destroys the options * following the token, the token must be added before coap_add_option() is * called. This function returns the number of bytes written or @c 0 on error. */ size_t coap_add_option(coap_pdu_t *pdu, unsigned short type, unsigned int len, const unsigned char *data); Change-Id: I1add40ab44e0e57e675f56db0211b12414dfa5ae Signed-off-by:Jaewook Jung <jw0213.jung@samsung.com> Reviewed-on: https://gerrit.iotivity.org/gerrit/14539Tested-by:
-
jihwan.seo authored
Change-Id: I2f1118de1217f61a8d0c7aa820bf27d0ef14773b Signed-off-by:
Dmitriy Zhuravlev <d.zhuravlev@samsung.com> Reviewed-by:
Joonghwan Lee <jh05.lee@samsung.com> Reviewed-by:
-
Oleksii Beketov authored
Fixed to comfirm compatibility in case if no PSK hint received on the client side. Change-Id: I2860188f966d8a71dbd834873f206b3ae78ec93e Signed-off-by:
Oleksii Beketov <ol.beketov@samsung.com> Reviewed-on: https://gerrit.iotivity.org/gerrit/14815Tested-by:
-
- 30 Nov, 2016 5 commits
-
-
hyuna0213.jo authored
remove unnecessary/duplicated log in connectivity abstraction to print the correct message. Change-Id: I2d0c4535bb4abc576d7e7130b0d38111f37cd3e0 Signed-off-by:
Mike Fenelon <mike.fenelon@microsoft.com>
-
Habib Virji authored
Removing this reduces ocbtstack from 180kb to 168kb in size. cJSON is not used anymore in the resource, except in json2cbor tool in the security module. cJSON is used in coap-http and simulator and scripts has been updated to include and build cjson. It was included mostly in sconscript. pmutility had helper function which was not used anywhere. smutility, psinterface, oicgroup all had function calling cjson but the function was not used anywhere in the code. Except psinterface and smutility there are no public API with cjson. The public API are not called by either sample or test code. The API are obsolete and using them will not yield any result and can crash the platform. Change-Id: Ifbf3384536394a26cd6b168dcd15285ef9ce08a7 Signed-off-by:
-
Ossama Othman authored
The TinyCBOR API requires that strings it allocates should be deallocated with the free() function. OICFree() is not an appropriate replacement in this case since it is conceivable the underlying memory allocator could differ. Change-Id: I48325c8fd41eaee156bf2da5e2f67b243114e3de Signed-off-by:
Ossama Othman <ossama.othman@intel.com> Reviewed-on: https://gerrit.iotivity.org/gerrit/14695Reviewed-by:
Thiago Macieira <thiago.macieira@intel.com> Reviewed-by:
-
bg.chun authored
provide public capi "OCCreateResourceWithEp" for mapping endpoint to resource and filltering request which come from unbinded endpoint. Change-Id: I7f273acdacff9116d99b60afb76f70fa48709341 Signed-off-by:
-
Randeep authored
Change-Id: If2b022d06a46c46cba1b3535c49ddcdbc794bf3c Signed-off-by:
-
