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>

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

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

Where only one filter was supported before, resource type filters must
be added one at a time through use of the OCDoResource() API, therefore
a call to OCDoResource() for every amend is req'd with this add-on.

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

Merge "Fix the name of the resource created in Arduino server to Light."

Merge "Fixed EntityHandlerRequest initialization to match other uses"

Merge "Fixed OCSerialization NVP Header Reset"

Added release notes for Dec 6th release.

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

Fixed OCSerialization NVP Header Reset

OCSerialization was not resetting the name-value pair header on error,
so this fixes that issue.  When searching for a NVP that doesn't exist,
cereal/RapidJSON will leave the iterator in a 'bad' place that can result
in not being able to find what we're looking for.

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

Fixed EntityHandlerRequest initialization to match other uses

Everywhere else we are using initializers rather than memcpy, so
this fix makes this one instance match the rest of the initializers.

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

Fix the name of the resource created in Arduino server to Light.

This is to bring resources in sync with Linux client example.
Now Linux client samples can interact with Arduino server.

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

Merge "Implement functionality to set PSK credentials for DTLS in OC stack."

Implement slow response feature.

This feature allows a resource entity handler to provide a "slow response"
to a client request.  That is, if a request is received but the server cannot
respond immediately (e.g. response data is not available), the server can
instead respond at a later time.  The same API is used for both immediate
and slow responses.

Updated collection's batch interface to aggregate responses of local resources
contained in the collection.  Note that the aggregate response only applies
to local resources, not remote resources.

This feature introduces a new C API, OCDoResponse and a new C++ API,
OCPlatform::sendResponse.  The C++ modifications also include a change to
the the signature of the applications entity handler.

Change-Id: Ic33ab0c7ff52481bed86e9a77319baa3813dd6cd
Signed-off-by: Doug Hudson <douglas.hudson@intel.com>
Signed-off-by: Yamin Al-Mousa <yamin.s.al-mousa@intel.com>
Signed-off-by: Sashi Penta <sashi.kumar.penta@intel.com>
Signed-off-by: Sachin Agrawal <sachin.agrawal@intel.com>
Signed-off-by: Sudarshan Prasad <sudarshan.prasad@intel.com>
Signed-off-by: Erich Keane <erich.keane@intel.com>

Implement functionality to set PSK credentials for DTLS in OC stack.

Uptil now DTLS module in OCStack was using PSK credentials which
were hard-coded. Added API which can be used by application to
provide device identity and PSK credentials.

Change-Id: I4df6db6ab70ee69917d0111f06a6504f3ce56ce2
Signed-off-by: Chris Mitchell <cmitch2@intel.com>
Signed-off-by: Sachin Agrawal <sachin.agrawal@intel.com>

Moved location of tinyDTLS from csdk to extlibs directory

External libraries are currently being stored under extlibs
directory and therefore moved tinyDTLS also there.

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

Merge "Implemented JSON Serialization using Cereal Library"

Implemented JSON Serialization using Cereal Library

Previously we were hand-mangling strings for JSON serialization and using
boost::property_tree for parsing.  Now we are using an open source library (cereal)
to do both sides in the C++ stack

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

Merge "Fix sending 3 notifications when creating a resource"

Merge "Fix the infinite PRESENCE_STOPPED message"

Fix sending 3 notifications when creating a resource

Implemented bind resource interface/type internal functions which
do not send the presence notificiations.

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

Fix the infinite PRESENCE_STOPPED message

When a client to subscirbes to a server, and when the server process is
exited, client is getting was getting infinte PRESENCE_STOPPED messages.
This commit fixes this issue. Introduced a new stack result
type : PRESENCE_TIMEOUT to indicate timeout of the server.
PRESENCE_STOPPED is used for graceful stopping of the server.

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

Merge "Adding PCF() wrappers to OC_LOG() calls that were missing in the stack."

Merge "Implementation of connectivity abstraction feature-Ubutu v0.1"

Implementation of connectivity abstraction feature-Ubutu v0.1

Connectivity Abstraction feature provides unified approach for multiple
Transports

Adding WIFI transport support on Ubuntu platform in the  v0.1 release
We followed Coding guidelines as per current base source.

Please read the Release notes v0.1 for more information.
Non Supported code is just made to print the message.

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

Merge "Removed Protocol Plugin developer guide document."

Merge "Fixed: bug where Header Option count would be zero"

Removed Protocol Plugin developer guide document.

Because uploading the developer guide document to the web page,
we no longer manage the document in the repository.

Change-Id: I5aeed10245ce1d255ce69dfd7a2255c5e9f9cdf9
Signed-off-by: junho13.lee <junho13.lee@samsung.com>

Fixed bug where InvalidParam exception thrown on re-observe

Observing a resource more than once isn't allowed.  However, we were
not clearing the "Observation Handle" on observation cancel.  This
patch resets the observe handle to nullptr when cancelling observation
so that we can re-observe after cancelling.

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

Fixed: bug where Header Option count would be zero

01.org IOT-76

Header options had a defect introduced attempting to fix
the 'zero' header option count value.  In the previous defect,
the stack would have an uninitialized header options count
when there were none.  The previous fix has been removed and
initialization has been done in the correct place.

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

[SSM] Adapt astyle code formatter

Problem: Follow coding guideline

How: Adapt astyle cpp formatter

Change-Id: I58ba77542f5fb3a935e1083a076957357599bc7e
Signed-off-by: Kim Jee Hyeok <jihyeok13.kim@samsung.com>

Merge "[SSM] Modify thread util and framework init method"

Merge "Supporting Java Plugin and making plugin manager as a library"

[SSM] Modify thread util and framework init method

Problem: Thread worker crashes while terminating.
         SoftSensor searching path must given at compile time

How:     Rearchitecture threading model
         Add XML descriptor based framework initialization.
           1. Search SoftSensorDescription.xml
              (previously high-context.xml) where executable located
           2. Or put search path using below init configration.

<SSMCore>
    <Device>
        <UDN>abcde123-31f8-11b4-a222-08002b34c003</UDN>
        <Name>MyPC</Name>
        <Type>PC</Type>
    </Device>
    <Config>
        <SoftSensorRepository>/usr/local/SSRepo/</SoftSensorRepository>
        <SoftSensorDescription>/usr/local/SSRepo/SSDescription.xml</SoftSensorDescription>
    </Config>
</SSMCore>

Change-Id: Id2c0746357d9e850e9e90016547b116412b758d2
Signed-off-by: Kim Jee Hyeok <jihyeok13.kim@samsung.com>

[SSM] Fix to call OIC platform destructor

Problem: There is no destructor caller

How: Call platform destructor when OIC platform manager destroying

Change-Id: I83b033494f3bb84e5341c21d7ec4051e110a9ff8
Signed-off-by: Kim Jee Hyeok <jihyeok13.kim@samsung.com>

[SSM] Fix model unregistration problem

Problem: If try to unregister registered model,
         unable to find registered model because of key mismatch.

How: Use proper key token to find registered model

Change-Id: Ib66242a4532e6a5fd8ced8e232fea26672347880
Signed-off-by: Kim Jee Hyeok <jihyeok13.kim@samsung.com>

[SSM] Remove global mutex

Problem: global mutex is not necessary
          while initializing platform.

How: Remove global mutex class and references.

Change-Id: I8972c6eb7d67dbd91c5c8638bec22fe83f83ef43
Signed-off-by: Kim Jee Hyeok <jihyeok13.kim@samsung.com>

Supporting Java Plugin and making plugin manager as a library

1) Cleaned unnecsssary files.

2) Changed plugin manager to support C/C++ and Java Plugin. Java plugin manager is not completed yet.

   This commit only have template code for Java.

3) Created plugin manager as a separate library(libpm.a) to be linked by application directly.

   And separated plugin manager implementation as shared library(libpmimpl.so) to support out-proc mode with same libpm.a in the future.

Change-Id: I5a80faa0546da1832e15a75dacc7f916faedd868
Signed-off-by: Lee Jun Ho <junho13.lee@samsung.com>

Adding PCF() wrappers to OC_LOG() calls that were missing in the stack.

By ensuring all calls to OC_LOG() wrap their strings with PCF(), we're
ensuring log statements don't take up program memory on lite platforms.

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

Added release notes for Nov 15 release.

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

Modify presenceclient sample

Modify presenceclient application to subcribe to MC address
without the need to discover a resource.

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

Merge "Support resource type for multicast subscription"

Active Discovey Resource Type Filtering - C++ SDK & App Changes only.

This change affects only the C++ API and it's sample app
"presenceclient.cpp".

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

Aligning name of resource between C Apps and C++ Apps.

This change to align the "LED" resource to be named "Light"
allows for inter-SDK application verification. This will help
ensure that both SDKs operate similarly, if not identically.

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

Add to C SDK sample app to subscribe to presence with or without filter.

This allows for end user to see the difference between a filtered
presence subscription and a non-filtered presence subscription. Also
added some minor logging changes.

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

Merge "Active Discovery Resource Type Filtering - C SDK & App Changes only."

Support resource type for multicast subscription

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

Active Discovery Resource Type Filtering - C SDK & App Changes only.

Applying required API changes and tying up a part of the functionality
required for implementing resource type filtering on presence
notifications.

Similar C++ SDK & App changes to follow this commit.

Change-Id: I9a8cd1a3e6e30c5de5788f593ef3efd2891b2f56
Signed-off-by: Omkar Hegde <omkar.m.hegde@intel.com>
Signed-off-by: Joseph Morrow <joseph.l.morrow@intel.com>

Make command line option for secure and observer

Implement command line option for secure resource and observer

Remove duplicate case to avoid confusion

Update app usage description

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

Implemented Secure resource support in C++ layer

Added ability for the C++ stack to properly parse/support the secure protocol URIs.

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

Modified OCInitUDP to support security.

Added a new flag to OCInitUDP to indicated whether to reuse the address or not.
Modified create_new_context API. It now takes the ipAddr[4] as input. This needs
to be passed further down to dtls_init.

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

Implement sending host port number for secure resources

Secure resources are hosted on a separate IP port. All the
discovery responses are send from unicast unsecure port from
resource server. Currently, when a client receives discovery response
it 'assumes' that the sending port is the port hosting the resource
too. This 'assumption' is NOT valid for secure resources. Therefore,
there is a need to send 'secure port info' for secure resources in
JSON discovery response.

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

Merge "Implemented libcoap's tinyDTLS interface"

Implemented libcoap's tinyDTLS interface

libcoap needs to encrypt/decrypt packets while sending on
secure port. This changeset implements interface for libcoap
to use tinyDTLS for creating DTLS sessions and performing
crypto operations.
This changeset is dependent on having tinyDTLS installed
under oic-resourc/resource/csdk directory.

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

Merge "Added support for multicast presence"

Added C++ Sample application using multiple threads.

C++ Sample application demonstrating the idea of running multiple
servers and clients running in multiple threads.

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

Merge "fix coding standard problem"

fix notification manager for m2

fix all code based on latest resource api.
fix notification folder name, following name convention.

Change-Id: Ic1eba69dd712428b4b0fae542d4acb005ad78a52

fix coding standard problem

fix all coding standard problem. This commit did NOT include logic changes .

Change-Id: I34f7a33374f137b8b641dff74e113b4941cb1391

fix things manager sdk for m2

Change-Id: I681378772f6b9dd120fd23abc124e66cef85c07e

Added support for multicast presence

Added OCMulticastNode data structure to store the nonce information of the server send
presence notifications. When a new notification comes to the client, client checks if there
is a multicast node, and then checks to see if the nonce is same or not. If same nonce, no call
back to client application is made. If nonce is different a call to client application is made.

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

(1) add platform overload api : registerResource(resourcehandle, resource), (2) add host in rsrc_t

Change-Id: I4809566e153eb3f488617adb7e044da869c1d88c

Removed Iotivity logo.

Until we get clearance with Iotivity logo, we are not going to use any logo.

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

Repo Merge: Modified makefile for the soft sensor manager

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

Repo Merge: Modified makefiles for the things graph manager

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

Repo Merge: Adding Arduino WiFi/Ethernet patches

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

Repo Merge: Move cJSON from oic-utilities to remove the dependency

Modified makefile for the cJSON move.

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

Repo Merge: Moving resource API down a directory

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

Updated Makefiles and CMakeLists.txt to point to resource, not oic-resource

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

Added snapshot of oic-service repository

The original commit hash in oic-service is 6e5c509395d8fa54ea4849b7065c0e2d0bad6b0d.
No attempt is made in this commit to update Makefiles to build.  Makefile updates
will happen in a separate commit.

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

Merge "fix darwin and ios build error"

fix darwin and ios build error

1) the logger implementation only consider arduino/android/linux
should add more platform in consider, such as darwin/ios ...

2) Without include <string>, there will be build error with xcode
compiler.

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

Merge "Updated libcoap to send messages using secure port"

Updated libcoap to send messages using secure port

Whenever a REST request is received by OIC Stack on secure
port, it needs to keep track of this fact for two reasons:
1) It allows upper layer to grant/deny access to secure resource
if the request was unauthenticated.
2) It also allows OC Stack to encrypt the response for a secure
request.
Since existing code was using a mechanism of send_flag to keep
track of various send mechanisms, it was modified to a bit mask
flag.

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

Merge "Removed 'resource implementation' of libcoap library For enabling security in libcoap, there is a need to modify methods which are being invoked from 'resource interface' of libcoap. Since libcoap's 'resource interface' is unused in OIC stack, it does not make sense to spend dev cycles in keep on maintaining that code."

Removed 'resource implementation' of libcoap library
For enabling security in libcoap, there is a need to modify methods
which are being invoked from 'resource interface' of libcoap. Since
libcoap's 'resource interface' is unused in OIC stack, it does not
make sense to spend dev cycles in keep on maintaining that code.

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

Merge "Fix wrong return value from OCInitUDP API"

Fix wrong return value from OCInitUDP API

Call to setsocketopt method was hiding the return value from bind
method call (in case bind API fails). Modified code to use the
return value from bind call for API return value.

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

Merge "Fixed bug where fridgeclient would immediately exit"

Merge "Modify C Sample Applications & solve memory leaks."

Merge "Fixed Iotivity logo."

Fixed Iotivity logo.

Since Iotivity is going public, the logo needs to match the correct logo.

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

Merge "Fix crash issue after observation"

Fixed bug where fridgeclient would immediately exit

When removing the parameter from the constructor, I left the
parends in, which causes it to be interpreted as a function rather
than as an object.  This results in the condition_variable never being
hit.  This returns functionality to forever-blocking like it was
before.

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

Fix crash issue after observation

I discovered a crash in simpleserver C++ that when being debugged,
it would crash on the observation AFTER the client had cancelled.
The cause was that the NumberRecvHeaderOptions sent was an invalid
character rather than 0!  This fix repairs the issue by ensuring that
the values are initialized correctly.

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

Merge "Release notes for OCT 31, 2014 release"

Release notes for OCT 31, 2014 release

Updated release notes based on review comments.
Change-Id: I85e9cf672aaa9e2dbda73e27a7aea9ec51b0aa4a

Adding example of a Simple Attribute use on a collection resource using
C SDK.

This involved forming proper JSON examples, as well as adding test
cases to the application as the implementation of test cases were not
fullfilled yet.

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

OCPlatform Object Removal

Removed dependency on the OCPlatform object.  We turned it into an internal-singleton(renamed to OCPlatform_impl so that we can use OCPlatform for the namespace, making it more consistent with previous code) with the expectation that people are to use OC::OCPlatform::<functionname> to control the API.
Change-Id: I3f0e13a27021f7454da68add93aaf562506e08ec

Modify C Sample Applications & solve memory leaks.

New Sample applications to demosntrate the basic functionality
of the stack. Added calls to free() to solve JSON related memory
leak issues.

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

Resolve name collision

Since these two files differed in their names only by case
it was not possible to check them both out at the same time
on a case insensitive filesystem.

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

Merge "roomserver/client updated to demonstrate simple attribute support in collections. Addressed review comments."

Merge "Adding C++ UB Unit Tests AND C++ UB Samples to buildScript.mk makefile."

Adding C++ UB Unit Tests AND C++ UB Samples to buildScript.mk makefile.

This change will be reflected by calling make on any 'Linux' "all" target
in the buildScript.mk makefile. This also contains refactor from UB & TB
to Rich & Lite, respectively.

Change-Id: Ie9c04c28c3f85bf5ffda29c6591cbd2eaeaac1e8

roomserver/client updated to demonstrate simple attribute support in collections.
Addressed review comments.

Change-Id: I4a054f1990954a9fd9dd3e98feddd204fd692676

Fixed server crash when we had less than MAX header options

Change-Id: I16a7a1c1049869bd2c26dcec903c2b3f82c11bfb

Added some string constants to cover exception cases
Note: I apparently messed up checking if this would compile/work :/  Build server is not working apparently.

Change-Id: I1b7eb33e4f9f1d184eb8c8488c0fe3052a1f1359

Added some string constants to cover exception cases

Change-Id: I102a7b136b6cc2d5c309e8c7984869aa51594e80

We wish to prevent multiple instances of OCPlatform from being accidentally created, so we make the copy-mechanisms private!
Patch 2: Fixed wording of comment in OCPlatform
Change-Id: Ic53816972426a29afc4764982a3ad712fa284945

Merge "Fixing the client.c in csdk/stack/test/linux to use OC_LOW_QOS instead of OC_NON_CONFIRMABLE. This is needed when we include unit tests in the build script."

Merge "Edited Arduino Makefiles and READMEs to describe how to ARDUINO_PORT."

Fixing many more regressions from String constants change.

Change-Id: Id1b6c0f5618c28720565e55354a73eade190f4a4