Merge "Update scons build relative documents"

Merge "remove DMR doc"

remove DMR doc

Change-Id: Icc34f18a66d87ae74748e496c3f01ff94ae84aa9

Merge "remove DRM doc"

remove DRM doc

Change-Id: Idd96a81cd79d1b6c54776c6b39cb080743107f8f

Update scons build relative documents

Change-Id: I5296365d941d876ec20252282f8bae1cb5d8b7ff
Signed-off-by: Caiwen Zhang<caiwen.zhang@intel.com>

Merge "Fix GroupAction API of Things Manager."

Merge "Fix an exception handling issue in doBootStrap() function of ThingsConfiguration class."

Fix an exception handling issue in doBootStrap() function of
ThingsConfiguration class.

In the case that a callback pointer of the function is NULL,
a return value for the function call will be OC_STACK_ERROR
rather than OC_STACK_OK.

Reference issue ID : [IOT-196]

Change-Id: Ieb280bc6e5317496b96342a0374129c1e692b415
Signed-off-by: jihun.ha <jihun.ha@samsung.com>

Fix GroupAction API of Things Manager.

Crash is occurred when some api of ThingsManager is called
with null ocresource or null actionset.
It will return OC_STACK_ERROR, if ocresource or actioset is null.

Fix [IOT-199], [IOT-200], [IOT-201], [IOT-202]

Change-Id: I1bf1070caaa4d06aad21aa26011474aae263a544
Signed-off-by: HyunJun Kim <hyunjun2.kim@samsung.com>

Merge "[service] Modify SConscript to build all services using scons"

Merge "[SSM] Fix scons android build failure"

[service] Modify SConscript to build all services using scons

Change-Id: I6a4ed93f7b3ac1c6fe40178d629fd22ae61093e2
Signed-off-by: jk13 <jihyeok13.kim@samsung.com>

Merge "Add readme file about the things manager."

Add readme file about the things manager.

Change-Id: Ia58badcc20ac83babaf3e656d2b36a41844bf0de
Signed-off-by: HyunJun Kim <hyunjun2.kim@samsung.com>

[SSM] Fix scons android build failure

1. Fix scons while build errors if choose android build

Change-Id: I78f6bccc1c27689a4bf847c602e207454f20450f
Signed-off-by: jk13 <jihyeok13.kim@samsung.com>

Add 'tizen' into the target OS list

In some case, it is required to know exactly the target OS is 'tizen'.
e.g. there are linux example applications and tizen example applications.
To get to know which example applications should be built, must
distinguish 'tizen' and 'linux'.

Change-Id: Icf056cba834770f129db3b822f452088bf2e290c
Signed-off-by: Caiwen Zhang<caiwen.zhang@intel.com>

Conditionaly enable android build

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

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.

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

Remove oic-utilities as a requirement

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

Fix service build error

1. fix protocol-plugin service build error
2. disable the building of the services which aren't updated

Change-Id: Ic9ff2c76610c236811de5c71cb98f03ae63b1db0
Signed-off-by: Caiwen Zhang<caiwen.zhang@intel.com>

Download gtest-1.7.0 rather than depending on it being installed

Making the user download and install gtest-1.7.0 is inconvenient for the
user.  Also, we do not have direct access to the build machine to update
how and where gtest is stored.  Downloading to the extlibs directory in
project source tree removes the need for developers to download and build
gtest and removes the need to configure the build machine.

Change-Id: I3b163c09ecdd9f12e8004a69dfa8dbcaf4ae5615
Signed-off-by: William Dieter <william.r.dieter@intel.com>

Added Release notes for Dec 20th release

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

Merge "Fixing delete issue in device discovery server sample"

Fixed Scons build Error.

  Before scons build is not working because of cpluff build and mqtt build.
  It is resolved.

Remove null check point cpluff.
  PluginManger will pass the argument to plugins later.
  Currently it is not needed yet.
  Comment out related code.

Modify the readme file for scons build
  Before readme file is for makefile.

Comment Out Notification build in the Scons script.
  Because of scons build error.

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

Merge "Replaced sprintf with snprintf in the sdk to avoid potential buffer overflows"

Merge changes Ie4233a5c,I4d7aba08

* changes:
  Ignore SCons generated files and directories.
  Removed execute permissions from non-executable files.

Updated sample apps to show use of multiple presence RT filters.

Updated C sample apps and C++ sample apps to display a simple multi-
resource presence use-case. This shows a few cases which automatically
generate presence notifications.

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

Merge "Make all multicast coap messages sent as Low_QoS/NON."

Make all multicast coap messages sent as Low_QoS/NON.

When a client in C++ is configured to use HighQoS,
then discovery request goes as CON to a multicast address.
However, responses/ACKs are coming from a unicast address.
This makes it impossible for clients to associate requests
with responses and stop retrying CON pdus.Now, CoAP layer is
making sure that all packets sent to multicast address are NON.

Change-Id: I41f351c5c9ce75fc4846d970514733bef5cdce61
Signed-off-by: Yamin Al-Mousa <yamin.s.al-mousa@intel.com>
Signed-off-by: Sudarshan Prasad <sudarshan.prasad@intel.com>

Merge "Fixed Warning in OCException"

Fixing delete issue in device discovery server sample

Earlier we were doing a new[] on device info and cleaning up using delete.
Changing "delete" to "delete[]".

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

[service] Update service documents.

Change-Id: Ifc39cbee3321a43c45c004099b5ba36cd4b115ac
Signed-off-by: HyunJun Kim <hyunjun2.kim@samsung.com>

Merge "Fix issues of Things Manager"

Merge "Add scons script and things manager samples."

Merge changes I9ccf4bec,Id22df7b5

* changes:
  [service] Update service documents
  [SSM] Add OIC namespace and cleanup description file

Merge "[SSM] change soft sensor description folder path"

Implementation of response of group action (for remote resources)

There is no response when resource client send message[POST]
to a group with GROUP_INTERFACE, so client cannot receive any message.

We provide logic about the aggregation of children' response.
Then, merged response will be delivered to client.

Additionally, regarding Sudarshan Prasad's comment of previous commit.
We change the definition of GROUP_INTERFACE.

Change-Id: I6c2550472d1e02cd2606317fcb1796e59e004b78
Signed-off-by: HyunJun Kim <hyunjun2.kim@samsung.com>

Merge "[SSM] Reference & Trackee Thing release for arduino"

[service] Update service documents

Change-Id: I9ccf4bec3ac95b9219f160a851186d431ff8b818
Signed-off-by: jk13 <jihyeok13.kim@samsung.com>

[SSM] Add OIC namespace and cleanup description file

1. Add OIC namespace at SSM interface header for application developers
2. Add description for scons build and run

Change-Id: Id22df7b59fd30d67a05703ccbafc845e94f05ff3
Signed-off-by: jk13 <jihyeok13.kim@samsung.com>

[SSM] change soft sensor description folder path

1. Change folder path
2. Set release folder for documentation

Change-Id: I2c307c813f87631122eec7e867845192739937b5
Signed-off-by: jk13 <jihyeok13.kim@samsung.com>

Fix issues of Things Manager

 - Bug fix
   : Implementation of Things Manager feature
     https://oic-review.01.org/gerrit/495
 - Resolve issues detected by Prevent (static analysis tool)

Change-Id: I6a06aa5a688c75384248638b83337e1d1b2dbd56
Signed-off-by: Soyoung Youn <sy01.youn@samsung.com>

Add scons script and things manager samples.

oic-resouce/Sconscript
 - Build for things manager SDK and all samples.

service/things-manager/sampleapp/
 - Build for all samples.
 - Sconscript

service/things-manager/sampleapp/linux/configuration
 - Sample application for Device Management.
 - Sconscript.

service/things-manager/sampleapp/linux/groupaction
 - Sample application for Group Action.
 - Sconscript.

service/things-manager/sampleapp/linux/groupsyncaction
 - Sample application for Group sync.
 - Sconscript.

Change-Id: I6c9975824a23deb5363b7157d64ad334bb11c915
Signed-off-by: Soyoung Youn <sy01.youn@samsung.com>

Merge "[SSM] Apply OIC presence feature"

Device Discovery - C & C++ SDK changes

Adding API for setting device information.
Currently supports filtering of device information based on device id and
device name only.
Updated C and C++ sample applications

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

Ignore SCons generated files and directories.

Change-Id: Ie4233a5c36efa09a905074b3ab5c100d96bd5033
Signed-off-by: Ossama Othman <ossama.othman@intel.com>

Removed execute permissions from non-executable files.

C and C++ source files, Makefiles and non-executable
script files (e.g. those without a #!/bin/sh on the
first line) should have not execute permissions.

Change-Id: I4d7aba082041fb4da79a1ec24009460c52ac7135
Signed-off-by: Ossama Othman <ossama.othman@intel.com>

Merge "Fix prevent bug in the PluginManager. Fix bug in the sample app."

Merge changes I21ded957,Ia1308e81,If6bccb8e,I6d548404

* changes:
  [SSM] Fix static analysis tool issues and 64bit build support
  [SSM] Update SDK Api
  [SSM] Remove document folder
  [service] Publish service documents

Fix prevent bug in the PluginManager.
Fix bug in the sample app.

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

[SSM] Reference & Trackee Thing release for arduino

1. Reference Thing code release for arduino
2. Trackee Thing code release for arduino

Change-Id: Ib05327af485289dba1b4a1d0ee2b143b5434ab15
Signed-off-by: min7.choi <min7.choi@samsung.com>

[SSM] Fix static analysis tool issues and 64bit build support

1. Fix code that detected by prevent(static analysis)
2. Fix code to support 64bit build

Change-Id: I21ded9571d2b4fc257991a534171e1931e6ba344
Signed-off-by: jk13 <jihyeok13.kim@samsung.com>

[SSM] Update SDK Api

1. Release Java API
2. Separate into cpp, java folder

Change-Id: Ia1308e814a4cb79575b11bc24ddce69376f2074a
Signed-off-by: jk13 <jihyeok13.kim@samsung.com>

[SSM] Remove document folder

Change-Id: If6bccb8e4cae72baf08b66c6febab8d40a8ac7bc
Signed-off-by: jk13 <jihyeok13.kim@samsung.com>

[service] Publish service documents

Change-Id: I6d5484042e8cef41403511fe206167ff30974fd8
Signed-off-by: jk13 <jihyeok13.kim@samsung.com>

[SSM] Apply OIC presence feature

1. Apply OIC presence check feature to Soft Sensor Manager
2. Add OIC presence feature to Thin Device Server Example

Change-Id: Iaa65942fb7bea5d0954cb65757ccb35bb2c12708
Signed-off-by: minjii.park <minjii.park@samsung.com>

Merge "Fixed build error"

Merge "Added code to send close_notify alert when OC stack shuts down."

Fixed build error

Due to a previous merge, build was failing when scons was used.
Signed-off-by:Sudarshan Prasad <sudarshan.prasad@intel.com>

Change-Id: I26ad37273f91e61bd4afb49c9c4010cde32cb3b9

Merge "OIC scons build patch for Yocto"

Merge "Implementation of Things Manager feature."

Merge "Implementation of Group action feature for Things Manager."

OIC scons build patch for Yocto

Function:
Adds Yocto as a build target for OIC. Code is
executed ONLY when TARGET_OS=yocto.
It WILL NOT affect a build for any other target.

Background:
Yocto cross-toolchains for platforms would have
to be fetched, installed and configured in their
standard way. This configuration is done by
sourcing in an “environment setup” script that
comes with the toolchain. scons is run after this
step.

Details:
The code in this patch reads the variables set
by Yocto’s script and: 1) Learns of tool names and
build flags (varies by target and arch.), 2) Finds
absolute path of these tools (yocto’s script adds
these to PATH) by looking in PATH, 3) Modifies the
scons construction environment to point to tools
+ all their required flags, 4) Switches TARGET_OS
to Linux so that it uses the Linux build configuration
for OIC, 5) Elects to preserve debug symbols (using -g)
so that Yocto’s build process can generate both
DEBUG and RELEASE packages for OIC (This is the
default recommendation of Yocto’s BitBake)

Additional notes:
-The pre-existing TC_PREFIX/TC_PATH framework is
not sufficient to support Yocto’s cross-toolchain.
-Supported architectures for the yocto target are x86
and x86_64. These have been tested to work for Intel
Edison and Minnowboard platforms.

Usage:
From a user’s perspective, there are no unexpected
steps.
1) source <yocto-env-setup-script>
2) …/oic-resource$ scons TARGET_OS=yocto <regular options>
For eg. “scons TARGET_OS=yocto TARGET_ARCH=x86 RELEASE=1”
The results of a build will now go into “out/yocto/x86/…”.
If step 1) hasn’t been done before step 2), the build will
fail with an error message.

Change-Id: I55efab20f9c174f46446e9d59e3fca6eef3df7f2
Signed-off-by: Kishen Maloor <kishen.maloor@intel.com>

Merge "Fixed usages of strtok to use strtok_r instead"

Replaced sprintf with snprintf in the sdk to avoid potential buffer overflows

Defect IOT-123 stated that unsafe API sprintf was previously used in SDK and
the recommendation was given to use snprintf rather than sprintf in order to
avoid buffer overflow.

As a result sprintf was replaced with safer snprintf in multiple sdk files

Change-Id: Ic739506f160176aaeb78927745ea67e61c1c0066
Signed-off-by: Yuliya Kamatkova <yuliya.kamatkova@intel.com>

Merge "Fixed OCCreateResource return type when empty URI is passed"

Merge "Improve Buffer Size in occoaphelper"

Merge "Fixed server crash on empty client payload"

Fixed Warning in OCException

Added case statements for two new conditions in OCException that
were forgotten when we implemented this in the past.

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

Fixed compile error on 'empty' in OCRepresentation.cpp

The patch for fixing the 'empty' representation object issue
was incorrectly manually merged when it (review 633) was split from
the previous C++ unit test review.

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

Merge "Fixed UnitTest crash when getting JSON from blank object"

Implementation of Things Manager feature.

Add feature for Things Manager

 - Subscribe Collection Presence
 - CRUD function for ActionSet
 - Finding group
 - Creating group
 - Join the specific group
 - Let the other devices to join the specific group
 - Leave from the specific group
 - Delete the group
 - Get all group information
 - send configuration information to multiple things
 - send diagnostics information(i.e. factory reset, reboot)
 - Get the configuration parameter list
 - Get the diagnostics functionalities list

Fix Segmentation fault.
 - Bug fix : "Implementation of Things Manager feature."
             https://oic-review.01.org/gerrit/495
             OCPlatform_impl.cpp, InProcServerWrapper.cpp

Change-Id: I5b44fbdb034c924df4787a30a4d9c86ce4704aff
Signed-off-by: HyunJun Kim <hyunjun2.kim@samsung.com>

Fixed warning about incorrect initializer.

The ip_mreq initialization was generating a warning.  This
corrects the initializer to properly initialize it as far as
the compiler is concerned.

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

Improve Buffer Size in occoaphelper

Previously, we had the BUF_SIZE set at 64. Changing it to be the
greater of MAX_URI_LENGTH and MAX_QUERY_LENGTH. So, if max length
needs to change, we need to change it only in ocstackconfig.

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

Implementation of Group action feature for Things Manager.

Add GROUP_INTERFACE and OC_RSRVD_INTERFACE_GROUP.
These interfaces are supported Group Action.

When DefaultCollectionEntityHandler is received an message
which Using GROUP_INTERFACE,
It is called BuildCollectionGroupActionJSONResponse.

PUT : Action Create and Delete
POST : Action Read and Execute

Change-Id: I14519fa85b5319c3e9df72fa27e85a5d8df6ea87
Signed-off-by: HyunJun Kim <hyunjun2.kim@samsung.com>

Merge "Changed plugin public setValue API to private."

Merge "There are Two modifications. 1. Restucturing Notification Manager Class   : Hosting feature is separated from Resource Manager Class into Hosting Handler, Hosting Interface, Hosting Config.     Corresponding Sample code modification. 2. Resource Hosting Control Point (in the Hosting Interface Class)   : Set up the Control Point for Resource Hosting.     User can control how to host Resource by timing and nodes"

Changed plugin public setValue API to private.

    Application developer shoud not change the plugin infomation.
    remove unnecessary code.

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

Merge "[SSM] Fix build script"

Merge "[SSM] Apply changed OIC API"

Merge "Fixed IOT-110--Crash on destruction"

Fixed OCCreateResource return type when empty URI is passed

Defect IOT-109: OCCreateResource in the C SDK previously allowed empty URI.

The defect was fixed: OCCreateResource returns error status OC_STACK_INVALID_URI
if empty URI string is passed

Note: There are other places in code where only NULL check is performed
for URIs however empty string check is not performed.

Change-Id: Ia23fd2be20677614f267c73160ebefc56a5e6d71
Signed-off-by: Yuliya Kamatkova <yuliya.kamatkova@intel.com>

Added code to send close_notify alert when OC stack shuts down.

When peers receive close_notify alert they clean up the DTLS
session state.

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

[SSM] Fix build script

1. Apply scons
2. Fix arduino makefile
3. Cleanup makefiles

Change-Id: I9f9c1597df21be96b4d40cfdd277649349dd742e
Signed-off-by: jk13 <jihyeok13.kim@samsung.com>

[SSM] Apply changed OIC API

1. Code cleanup with formatter
2. Change code to use changed OIC API style

Change-Id: I9f52091d62022575acfdd9dc97d19db361239c4e
Signed-off-by: jk13 <jihyeok13.kim@samsung.com>

There are Two modifications.
1. Restucturing Notification Manager Class
  : Hosting feature is separated from Resource Manager Class into Hosting Handler, Hosting Interface, Hosting Config.
    Corresponding Sample code modification.
2. Resource Hosting Control Point (in the Hosting Interface Class)
  : Set up the Control Point for Resource Hosting.
    User can control how to host Resource by timing and nodes

Change-Id: I627eaff40f6a5c19a4297cafcf9ea9e3c739d520
Signed-off-by: Heewon Park <h_w.park@samsung.com>

Modify PPM makefile for using shared base library

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

Fixed IOT-110--Crash on destruction

A shared_ptr loop in callbacks for listen and a server destructor
were causing the stack to crash upon destruction of the Platform_impl.
Platform_impl would decrement the ClientWrapper shared_ptr (which would
still be >0 due to the callback having one).  When the OCStop was cleaning
up its resources, it would delete the client context, decrementing the
ClientWrapper ptr completely, resulting in OCStop being called inside
itself.

This bug was fixed in a few ways (each which are important, but would
have fixed this independently).

1- Changed OCStop to protect itself from re-entrance by using a guard-bool
to return immediately if it is being called as a child.

2- ListenContext now contains a weak_ptr to the clientWrapper instead of
a shared_ptr.  This makes it not destruct the clientWrapper upon callback
cleanup.

3- ClientWrapper no longer calls OCStop in the destructor when in BOTH
mode.  This is done to be consistent with the OCInit call in the constructor
as well as to correctly give the ServerWrapper ownership of the Init/Process/
Stop functionality in the BOTH case.

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

Fixed usages of strtok to use strtok_r instead

strtok is not thread safe and has a global state variable that
have serious side effects.  If any consumer of ours was to use
threads and strtok they could very easily corrupt the state on us
(or us on them!).  This fix replaces it with the non-side effect
POSIX version, which is strtok_r.  Note that in occollection.c
we are already using strtok_r.

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

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>

Fixed UnitTest crash when getting JSON from blank object

Generating a JSON representation from a blank object is an error
condition, so we were crashing.  The unit tests validated this
situation, so this fix returns the correct value in that situation.

Additionally corrected a comment highlighted in a previous review.

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

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

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>

Fixed server crash on empty client payload

Server would crash if given a blank string, so this will prevent
that from happening.  The resource request won't contain resources,
since it wasn't sent any, but that is a valid condition

Change-Id: I04a8ebbe0174f1025d924503ac66fad6c37e0582
Signed-off-by: Erich Keane <erich.keane@intel.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)"