Updated Iotivity to compile DTLS modules using SCons

Added Scons script for compiling security samples. Also, updated
CA Scons script to compile connectivity_abstraction.so with DTLS
when scons is invoked with WITH_DTLS=1. Also updated security samples
to use updated OCDoResource API with connectivity_type argument.

Change-Id: I1561266dda1c0ff88e1ad3684a1921b44f12839d
Signed-off-by: Sachin Agrawal <sachin.agrawal@intel.com>

Fixed multiple declarations issue

Thanks to Sam for finding the issue.

Change-Id: I357f83c043dc9aa5a7c0f8513eddf18514188269
Signed-Off-By: Sudarshan Prasad <sudarshan.prasad@intel.com>

Fixed build error when scons is used.

Added required LIBS in the SConscript to get through the build errors.

Change-Id: Ida0a11fd83609ee0aa40db64ffa05efd437abd1d
Signed-Off-By:Sudarshan Prasad<sudarshan.prasad@intel.com>

Merge "Soft Sensor Manager Build Error fix" into connectivity-abstraction

Soft Sensor Manager Build Error fix

Change-Id: Ib65f912526f3f33b764f691209b51e7b22181f44

Merge "Merge from master to CA branch" into connectivity-abstraction

Fixing Token Length issue in HandleCARequests

This change also enable Device Discovery.

Signed-off-by: omkar <omkar.m.hegde@intel.com>
Change-Id: I0dfb51dcdba1dbf2dc8f70d71bbaefe79bac1a37

Merge from master to CA branch

Change-Id: If6c0c7db4d8212ce74e523ca03c0ebd7eca024d1
Signed-off-by: Sudarshan Prasad <sudarshan.prasad@intel.com>

Fix to copy the header options from CA to Stack correctly.

Change-Id: I203333fc30eba89a8f56a7af1652cb96bd172c21
Signed-off-by: Sashi Penta <sashi.kumar.penta@intel.com>

Fixes to send the correct sender info when a presence notification is sent

Change-Id: Id903483a5f329692fc57275076af2c36d0d1adf0
Signed-off-by: Sashi Penat <sashi.kumar.penta@intel.com>

Merge "Enabling Multiple Interfaces in CStack." into connectivity-abstraction

Merge "[CA-Integration] Added code to get connectivity type from C to C++ layer" into connectivity-abstraction

Merge "enable C++ presence samples" into connectivity-abstraction

[CA-Integration] Added code to get connectivity type from C to C++ layer

Change-Id: I5afd5269595c5d45b355e10e6c9400f66dc74ef9
Signed-off-by:Sudarshan Prasad<sudarshan.prasad@intel.com>

Enabling Multiple Interfaces in CStack.

The stack is initialized with WIFI & ETHERNET
Discovery Messages are sent over WIFI and ETHERNET
CA_LE is not supported

Signed-off-by: omkar <omkar.m.hegde@intel.com>
Change-Id: Id7ec568f2d2effb45727c75958288de4690c9343

enable C++ presence samples

Change-Id: I51aadf193b504c3bc88e12544f19f63d636a136b
Signed-off-by: Sashi Penta <sashi.kumar.penta@intel.com>

[CA Integration] Pass connectivity type on which response is received

Change-Id: I2f8377ca88a79f9502465a40c5003505e5001ec6
Signed-off-by: Vijay <vijay.s.kesavan@intel.com>

Merge "[CA-Integration] Initial changes to enable CPP stack." into connectivity-abstraction

[CA-Integration] Initial changes to enable CPP stack.

Change-Id: I10cb4cb26782008b30a69d4a5ab49ec6b7c74b5f
Signed-off-by: Sashi Penta <sashi.kumar.penta@intel.com>

Enable log for csdk samples

Change-Id: I1344484e918ef3eb54a8a26d2ecc225a1172d4bc
Signed-off-by: Sashi Penta <sashi.kumar.penta@intel.com>

Initial Push to Integrate Presence With CA branch.

Filtering based on rt and TLL changes are needed

Signed-off-by: omkar <omkar.m.hegde@intel.com>
Change-Id: Ie7c02f7c96fa9feb3622dd969a121a46d06b0020

Implement ServerID parsing and sample app

This changeset will correctly parse the Connectivity abstraction
ServerID and provide it as a unique identifier to the client implementer.
This change also includes a sample app that shows off this functionality.

Note: this change was brought from https://oic-review.01.org/gerrit/#/c/624

Change-Id: I0151744d3481da806f5345c0798f7b951fb91394
Signed-off-by: Yuliya Kamatkova <yuliya.kamatkova@intel.com>
Signed-off-by: Sudarshan Prasad <sudarshan.prasad@intel.com>

Fix build issue

Change-Id: Ie0d491d47417db82cf0ee2511e86efc6079c83be
Signed-off-by: Sashi Penta <sashi.kumar.penta@intel.com>

Merge "[CA Integration] Updated C sample apps to accept connectivity type param" into connectivity-abstraction

[CA Integration] Updated C sample apps to accept connectivity type param

Change-Id: Ibb6ed2142e9f26981538c386ff38eeeca91f4f8e
Signed-off-by: Vijay Kesavan <vijay.s.kesavan@intel.com>

Merge "[CA-Integration] Added connectivity type param in APIs" into connectivity-abstraction

Merge "scons changes for connectivity-abstraction branch" into connectivity-abstraction

scons changes for connectivity-abstraction branch

Change-Id: Ib433a03454ac543cfcf189e596778414377ffe2a
Signed-off-by: Sashi Penta <sashi.kumar.penta@intel.com>

[CA-Integration] Added connectivity type param in APIs

Signed-off-by: Sudarshan Prasad <sudarshan.prasad@intel.com>
Change-Id: I368885126b5db6182708627312776b9d6da1f7f6

[CA-Integration] Support query parameters

Change-Id: Idc539fdac6f0424c8c8e33defb99646fe355a879
Signed-off-by:Sudarshan Prasad <sudarshan.prasad@intel.com>

[IOT-209][CA Block] CASendRequestToAll from client changes NON-CON messages as CON on server side - fixed

Signed-off-by: Abhishek Pandey <abhi.siso@samsung.com>
Change-Id: Ibbd93ffb758e512bb77181fba71d5d02c05de0d8

Merge "Updates occlientbasicops to include (resource, SID) detection." into connectivity-abstraction

Merge "[CA-Integration] fix for presence." into connectivity-abstraction

Updates occlientbasicops to include (resource, SID) detection.

I've updated the occlientbasicops demo to show that
previously-observed resources can be detected using a
combintation of the resource and SID fields to demonstrate
that a client can see if a given (resource, SID) pair has
been encountered before.

Note: This change was moved from oic-review.01.org/gerrit/#/c/634

Change-Id: Ia2531f3c86773aeac177eb2b063777aa0365a2c4
Signed-off-by: Yuliya Kamatkova <yuliya.kamatkova@intel.com>
Signed-off-by: Sudarshan Prasad <sudarshan.prasad@intel.com>

Merge "Remove the dead code" into connectivity-abstraction

Merge from master to connectivity-abstraction branch

Change-Id: Ib27785b3f8681b37885a94e608650eed871f9051
Signed-off-by: Sudarshan Prasad <sudarshan.prasad@intel.com>

[CA-Integration] fix for presence.

One of the fix for presence.
Code to replace method type with REST_GET for REST_PRESENCE is moved up into common code.

Change-Id: Ib55d8536f41622649abd3c8d7df024590b6eb9c1
Signed-off-by: Sashi Penta <sashi.kumar.penta@intel.com>

Merge "Fixed bug in array navigation in CA Adapter DTLS module" into connectivity-abstraction

Remove the dead code

Change-Id: I564a957a8daa1ce7df92c135f5f914b014103b81
Signed-off-by: Sashi Penta <sashi.kumar.penta@intel.com>

Fixes for scons builds

1) Remove oic-utilities as a requirement

2) Fail the build if one step fails

Setting this option in the bash script will cause the
script to terminate if any of the commands returns a
with non-zero value.  This is necessary for the build
server to understand when a build is failed.  Without this
flag, the script will exit with the status of the last command,
which in this script is "echo" so it will be 0, regardless if
a scons build step failed.

Ideally we would try each step and then report the failures
at the end, set -e will do the job until that can be developed.

3) Conditionaly enable android build

4) Correct a build issue in occoap.c with typecasting

Change-Id: I2711db6ff0814cff64b170cba20455e8ef52f905
Signed-off-by: David Warburton <david.warburton@intel.com>

Updated security sample apps to add "/" in front of resource URI.

Gerrit Review 712 in 01.org updated CA layer to mandate "/" in front
of resource URI's. Current change updates security sample apps
to co-exist with 712 change.

Change-Id: I133eeab9092aac9d74f536065f5e10e91f621a4e
Signed-off-by: Sachin Agrawal <sachin.agrawal@intel.com>

Fixed bug in array navigation in CA Adapter DTLS module

If there are multiple packets in the cached array list, only first
packet is send. Updated code to continue sending other packets.

Change-Id: I124e6b9f9477e0cdf9047b410acc11181f1f811e
Signed-off-by: Sachin Agrawal <sachin.agrawal@intel.com>

Merge "[CA-Integration] Use 5298 port for MC query since CA uses 5298 as MC port" into connectivity-abstraction

CA Integration: Adding observe cancellation support

Change-Id: Ib34b27fac26de63ccee75f03dc336f23ff3303a0
Signed-off-by: Vijay <vijay.s.kesavan@intel.com>

[CA-Integration] Use 5298 port for MC query since CA uses 5298 as MC port

Change-Id: Id29c3c4fa52e3f1b4c49fb1652e16cc6485d951c
Signed-off-by: Sashi Penta <sashi.kumar.penta@intel.com>

Merge "Removed a printf statement in ocstack.c" into connectivity-abstraction

Integrated CA layer and RI layer for DTLS security

Integrated OCSecurityConfig data structure with CA DTLS implementation.
Also updated RI layer to use security data structure variables from CA.

Change-Id: Ia708c8a61f3a8e56442c7acfa927cf2512024452
Signed-off-by: Sachin Agrawal <sachin.agrawal@intel.com>

Removed a printf statement in ocstack.c

Change-Id: I564b52a57d0c676b1baaf3150df3d2594d5662f6
Signed-off-by: Sashi Penta <sashi.kumar.penta@intel.com>

[CA-Integration] Fix Get working

We were not setting the numOPtions correctly for operations other than observe.

Change-Id: Iff3dd8e31d1278429ab1e42e97333eacd684190b
Signed-off-by: Sashi Penta <sashi.kumar.penta@intel.com>

[CA-Integration]: Temporary fix for stack smashing and fix support for "/"

Change-Id: I54b9e4f8b51981094c8d78145510d3e865413f58
Signed-off-by: Sashi Penta <sashi.kumar.penta@intel.com>

Merge "[CA] [All] URI support , Optimization , License verification" into connectivity-abstraction

[CA-Integration] : Add support for sequence number for observe

Change-Id: I776308ad20911da52ca408c1cbca406bf9747353
Signed-off-by: Sashi Penta <sashi.kumar.penta@intel.com>

Merge "Fixed return value bug in CAGetNetworkInformation API" into connectivity-abstraction

Add support OCNotifyAllObservers and OCNotifyListOfObservers

Change-Id: If878e1a2da88b364ecf9601d844d2caf3d5ce1ed
Signed-off-by: Sashi Penta <sashi.kumar.penta@intel.com>

Registering observers in the stack

Changes made to store information regarding remote end points
for observers to send subsequent notifications.

Change-Id: I46c15b32919c1b1edd7297ace9676fdeab44808b
Signed-off-by: Yamin Al-Mousa <yamin.s.al-mousa@intel.com>

CA Integration: Sending observe register/deregister request from client to server using CA header options

Change-Id: I78ae3d916607ef994cb8d042611f7771b1ada501
Signed-off-by: Vijay <vijay.s.kesavan@intel.com>

Fixed return value bug in CAGetNetworkInformation API

When some of the network interfaces are not UP/RUNNING, res is set to
an error value and this error value is returned from the API.

Also updated Linux sample application which was in-correctly allocating
memory to the buffer provided to CAGetNetworkInformation API.

Change-Id: I65bcbe9b91716309b8c152b50405fd71c56ba7b3
Signed-off-by: Sachin Agrawal <sachin.agrawal@intel.com>

Fixed bad memory allocation, which was causing corruption

Previously the stack was allocating based on a variable that was
never actually set, and despite that was likely allocating based on
the incorrect data.  This patch ensures that the allocation is
based on the actual size of the string.

Change-Id: Ie2feafe665b870919e17222dade5548794b6ce67
Signed-off-by: Erich Keane <erich.keane@intel.com>

[CA] [All] URI support , Optimization , License verification

1) Updating the "/" support
      Now "/" is madatory to send the request
2) Removing glib header files and updated read me file to downloaded separately
3) Optimization of WiFI adapter in arduino platform

Signed-off-by: ashok.channa <ashok.channa@samsung.com>
Change-Id: Iabda06b0ac592b89adbc9870d8c78faccdfe3a12

CA Integration: Use only non-confirmed messages for initial integration

Change-Id: Ibe2d3e18b4dca1230a2be1df69398d3687b121a0
Signed-off-by: Vijay <vijay.s.kesavan@intel.com>

CA Integration: Enabling PUT, POST, DELETE operations

Change-Id: Ifb32391c176bcc03f3c1372113f913bc5cc16bd4
Signed-off-by: Vijay <vijay.s.kesavan@intel.com>

CA Integration: Invoking application CB when response is received.

Mods to support sending server responses with CASendResponse
Rebase

Change-Id: I4925abf92af0c1ffe286e03bf233a7b08f8d043a
Signed-off-by: Vijay <vijay.s.kesavan@intel.com>
Signed-off-by: Doug Hudson <douglas.hudson@intel.com>
Signed-off-by: Erich Keane <erich.keane@intel.com>
Signed-off-by: Sashi Penta <sashi.kumar.penta@intel.com>

CA Integration: Enabling OCDoResource to send messages

Change-Id: I289340c8dc05cebe4bc1dfc2e7acdf73decc536b
Signed-off-by: Vijay <vijay.s.kesavan@intel.com>

Integrate OCInit and OCProcess functionality with CA

Change-Id: Ib11fc68c17f8f135e57e99c452a4100d84677fca
Signed-off-by: Sashi Penta <sashi.kumar.penta@intel.com>
Signed-off-by: Vijay <vijay.s.kesavan@intel.com>

Add required compiler flags for CA integration

Signed-off-by: Sashi Penta <sashi.kumar.penta@intel.com>
Change-Id: Ib5ef49afa6fbb729b7cd3208e6522af859c5b964

[CA] [All] Klockwork , Prevent , Memory leaks fixed

Updating the BLE Android and memory leak fixes
Signed-off-by: ashok.channa <ashok.channa@samsung.com>
Change-Id: I5f27f69729a949d6111dd096d72a484e87bc89a3

API change , Retransmission Callback on expiry , remove glib source for dynamic linking

Improvement of following functionality
1) API change - CACreateRemoteEndpoint to allow connectivity type for Remote Endpoint formation.
2) Retransmission timeout callback on Response Callback if number transmissions are expired.
3) Android BLE connectivity updation.
4) Removed glib library source and including the library as dynamic library.

Signed-off-by: ashok.channa <ashok.channa@samsung.com>
Change-Id: I37a79d4a88219fd4aa81993444cff68c0296b568

Merge "Implementation of following functionality" into connectivity-abstraction

Merge "Fix compiler warnings" into connectivity-abstraction

Implementation of following functionality

1) DTLS support for ETH and Tizen platform
2) Removing SUDO from make files
3) Optimization of WIFI , ETH adapters for all platforms
4) Adding single thread header files for consistency with source files.

Signed-off-by: ashok.channa <ashok.channa@samsung.com>
Change-Id: I8a92b0ef38b16818c740f3e95aad5ce1e65741a4

Implementation of connectivity abstraction feature Release v0.61

Connectivity Abstraction feature provides unified approach for multiple
Transports
Supported Features :
Ubuntu ( WIFI-DTLS, ETH) - Multi Thread
Tizen (WIFI, BT, BLE) - Multi Thread
Arduino( WIFI, ETH) -Single Thread
Android( WIFI, BLE) - Multi Thread

Please read the RELEASE v0.6 @csdk\connectivity for more information.
Signed-off-by: ashok.channa <ashok.channa@samsung.com>
Change-Id: I57c81845725f255b43829b9f9e8fe62e8d7b8517

Fix compiler warnings

Signed-off-by: Sashi Penta <sashi.kumar.penta@intel.com>
Change-Id: I70f105c4703045b32b84fa51cd8fb86ea6518d91

Moved all presence functionality into the stack layer from coap layer.

This task was initiated so that observe parameters can be polled from
whatever new(er) transport layer we decide to use in the future and then
specific presence parameters can be determined once we have those observe
parameters. This effectively causes all presence related work to be done
in the stack layer, while observe related parameters are transferred OTA
and presence parameters are simply extrapolated after the fact.

This changeset correlates to Rally User Story: US2195 "As an Iotivity
client and server developer, all presence related functionality in the
OCCoap layer is moved to the stack layer."

Change-Id: Ibb784946fe233e17834136ed96ebed3436372742
Signed-off-by: Joseph Morrow <joseph.l.morrow@intel.com>

Merge "Implementation of connectivity abstraction feature Release v0.5" into connectivity-abstraction

Merge branch 'master' into connectivity-abstraction

Conflicts:
resource/csdk/makefile

Merge "Update build scripts"

Merge "Added Plugin Manager Configuration Feature  - application developer can modify plugin manager configuration(xml) like plugin path, max number of plugins."

Merge "Updated the .gitignore"

Updated the .gitignore

Updated the gitignore to properly ignore things in the new folder
format, as well as added klocworks and some other missing ignores.

Change-Id: I75e7ac8497ae5a51f9bab00f90dc025190d68960
Signed-off-by: Erich Keane <erich.keane@intel.com>

Merge "Fixed need of implementers to include cereal in their build path"

Implementation of connectivity abstraction feature Release v0.5

Connectivity Abstraction feature provides unified approach for multiple
Transports
Supported Features :
Ubuntu ( WIFI , ETH) - Multi Thread
Tizen ( WIFI ,BT,BLE) - Multi Thread
Arduino( WIFI,ETH) -Single Thread
Android( WIFI,BLE) - Multi Thread

Please read the RELEASE v0.5 @csdk\connectivity for more information.

Signed-off-by: ashok.channa <ashok.channa@samsung.com>
Change-Id: Ic27f1d7c7aaa9f2d72883b94ab2b0a73e3aaf073

Added Plugin Manager Configuration Feature
 - application developer can modify plugin manager configuration(xml) like plugin path, max number of plugins.

Added More Plugin Manager APIs
 - int getState()
   getState API can get plugin's state
 - int rescanPlugin()
   Rescan the plugin folder

Removed Unnecessary Plugin Manager APIs
 - std::string getSupportedType(void)
 - int addSupportedType(const std::string rscType)

Disabled Auto Plugin Detection Feature
 - In previous version, PPM used iNotify to detect new Plugin detection.
   However, iNotify have platform porting issue and heavy. We will find better way in next version.

Change-Id: I0efac20efbc39b3fb0b2bcc1a34685acad1c7517
Signed-off-by: heesung84.kim <heesung84.kim@samsung.com>

Merge changes I56f29eac,Ic7530d44

* changes:
  Fixed usages of memset(..0..) to use Calloc/init'er
  Added OCCalloc functionality and Fixed Malloc Tests

Merge "Send presence notification on stopping presence.(Multicast)"

Merge "Fixed klocwork issues in C++ Stack"

Merge "Fix klocwork issues in C samples"

Fixed need of implementers to include cereal in their build path

Previously we did a bit of implementation in the OCRepresentation.cpp
file because we thought we needed to (due to Template confusion).

It has been realized that this isn't necessary, so this fix removes that
need.

Change-Id: I5e022fe37b822eae83e164686c979ac6f474135c
Signed-off-by: Erich Keane <erich.keane@intel.com>

Fixed klocwork issues in C++ Stack

Ran klocwork 10 and fixed the issues that were highlighted.

fridgeclient:
return values from constructResourceObject weren't checked
for null.  Additionally, variables sent to getValue uninitialized.

presenceclient: presenceHandle used uninitialized.

threadingsample: variables sent to getValue uninitialized.

OicJsonSerializer.hpp:
Removed dead code that had a large number of klocwork issues
Methods that didn't return despite having a return type.

InProcClientWrapper.cpp:
Contexts deleted in cases where they aren't passed to the C stack.

InProcServerWrapper.cpp:
entityhandlerRequest was used without being checked for null

Many samples:
Replaced while(true) with a condition_variable
Ensured 'main' returned 0.  Despite being optional, KW shows this as
a warning
incorrect 'main' prototype (argv specified array length 1!)

Out of the 40+ issues found of all types, the following were skipped
due to being in an external library:

Review:
CWARN.MEM.NONPOD (4 issues)

Warning:
CL.MLK (2 issues)

NPD.FUNC.MUST(1 issue)

Change-Id: I1e9fa222cc27352075a1b8f5d53787bd3020b525
Signed-off-by: Erich Keane <erich.keane@intel.com>

Fixed usages of memset(..0..) to use Calloc/init'er

Our code has a number of cases where we declare a struct then
immediately memset to all 0.  This uses the initializers which
are less prone to failure and are faster due to compiler optimizations.

Additionally, we have a number of cases where we OCMalloc a
struct, then immediately zero it.  This fix uses the new OCCalloc
in order to save unnecessary memset and reduce potential for accidental
error.

In a few of the cases, older compilers didn't handle the {0} initializer
correctly and gave errors (Arduino Mega).  Because of that, a few structs
were reordered to compile for that platform.

Change-Id: I56f29eac68240abea2fb8a69d888a09d61d4488a
Signed-off-by: Erich Keane <erich.keane@intel.com>

Fix klocwork issues in C samples

Scan klocwork and fix all issues in c samples that were highlighted.

Change-Id: Id018abfd380c8723842a24f20c8f781dca177458
Signed-off-by: Sakthivel Samidurai <sakthivel.samidurai@intel.com>

Merge "[SSM] Modify repo init method and fix Resource data reader"

Merge "[SSM] Reflect current OIC api style and add Tizen Inproc Sample"

Merge "[SSM] Implement Inproc feature for Linux platform"

Merge "Fix Unicast Presence Error"

Send presence notification on stopping presence.(Multicast)

Previously notifcation was not being sent on presence stopped.
Adding a check in case of multicast presence for presence stopped.

Change-Id: I774295b89fef607a3d1936d26071d79c41ec96b5
Signed-off-by: omkar <omkar.m.hegde@intel.com>

Fix Unicast Presence Error

Fix for IOT-95. Previously we had missed a "+1" when mallocing a char *

Change-Id: I96021ddce22c2fa7e3afa8f8d10e6da10b47ebbc
Signed-off-by: omkar <omkar.m.hegde@intel.com>

Added OCCalloc functionality and Fixed Malloc Tests

The OCMalloc tests didn't seem to be running correctly, so
this fixes that.  Additionally, added OCCalloc functionality so
that we can replace 'memset 0' in a number of places in the C Stack.

Change-Id: Ic7530d442c146724e91ec12646a4363de63f6874
Signed-off-by: Erich Keane <erich.keane@intel.com>

Improve Presence Callback in C++ SDK

Passing the host address information to the application on receiving
presence Notification. Based on IOT-91.

Signed-off-by: omkar <omkar.m.hegde@intel.com>
Change-Id: Ie6bc5e20c71e14e2745da435a65de874d6f28742

Merge branch 'master' into connectivity-abstraction

Conflicts:
resource/csdk/makefile
resource/csdk/stack/include/internal/ocresource.h
resource/csdk/stack/include/internal/ocstackinternal.h
resource/csdk/stack/src/ocstack.c

Change-Id: Ia681bc1b72f2611a0c51c36a58451bd409e1ddd1

Merge "Added support to allow more than one presence resource type filter."