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>

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

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

Edited Arduino Makefiles and READMEs to describe how to ARDUINO_PORT.

This fixes an issue that was stopping folks from installing to an Arduino
Due, but not Arduino Mega.

Patch 2: Fix mistype. I wasn't using the ARDUINO_PORT variable.

Change-Id: I8249077378a47cdedc721c5f3e8867c13661a183

Merge "Fixing a regression from the StringConstants change, we'd replaced the Property key with a different inadvertantly."

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.

Change-Id: Ib6effb0dd54a126f14e4efdef257693c69a767e2

Fixing a regression from the StringConstants change, we'd replaced the Property key with a different inadvertantly.

Change-Id: I8196194b860fc75c6f7d8f0bddebde5fdf379e87

Fix C++ Unittests build issue.

Change-Id: I82de00e976ebcf12b6f508e6fb6a222b6cd75e1f

Merge "Updated buildscript.mk to include build of TB unit tests and samples."

Merge "Patch 1: Unittests for C++ Stack: 1. Seperate directory for C++ unit tests. 2. tests.cpp file contains unit tests. 3. README explaining to create local.properties to indicate the location of gtest. 4. makefile to build unit tests with google testing framework. 5. One good sample unit test demonstrating the use of unit testing. Patch 2: Addressed review comments."

Merge "String Constant changes"

Merge "Reverted README back to instructions for buildScript.mk. As per Yamin, he simply acciddentally converted to something else.  He did not intend to push any changes to README.  I'm not sure how this change went through code review on Gerrit."

Updated buildscript.mk to include build of TB unit tests and samples.

I oops'd and some how overlooked this with the 'all_dev' target.

Patch 2: Swapping spots with parent. (ie. Switching parent-child dependency
between this and Change-Id: I1d20b09a095b320971765733b4343a7b781da1d9)

Patch 3: Bailing out on unit tests! The unit tests are rubbish. They
are wayyyyy out of date and WILL NOT build. THe unit test target is/was
pointing to some sample app by Jesse (that i presume was an accident
because it was put into the unit tests directory). This means we have NO
unit tests. This is bad.

Patch 4: Rebase onto master.

Change-Id: Ia9ca015a09bc3408c3ced99eb0432e0fb0814a12

Reverted README back to instructions for buildScript.mk. As per Yamin, he
simply acciddentally converted to something else.  He did not intend to
push any changes to README.  I'm not sure how this change went through
code review on Gerrit.

Change-Id: I3e9de7f5690fc129873f8e73eac1b1b78ba17fde

Patch 1:
Unittests for C++ Stack:
1. Seperate directory for C++ unit tests.
2. tests.cpp file contains unit tests.
3. README explaining to create local.properties to indicate the location of gtest.
4. makefile to build unit tests with google testing framework.
5. One good sample unit test demonstrating the use of unit testing.
Patch 2: Addressed review comments.

Change-Id: I6c0ec8af4f7fd13ce9bc8a56b7634074ac161b5f

String Constant changes

Change-Id: Ib90cc60fdd5c2ee44a697249af448e07116d75e8

Added commentary to the public API referring to new functionality of
notifications when a resource is deleted or when a server is shutdown
gracefully.

Patch 2: Edited commentary to OCStop() as per Erich's review.

Change-Id: I0b71bd3245072ac8f7cf9f67dff66b3d8b501355

Merge "Fix build issue for simpleserverHQ"

Merge "Release notes for Oct 14 release. Patch2: Incorporated review comments."

Fix build issue for simpleserverHQ

Change-Id: I54f2b45968559c0caa052bd53b1a900fd3178fe1

Release notes for Oct 14 release.
Patch2: Incorporated review comments.

Change-Id: Ieff471d87a8eeffe0e33768f9f70951d69e0adb7

1. C++ POST updates to include new URI and status for CoAP.
2. Now C++ Entity Handler also return OCEntityHandlerResult
3. Fix build issue.

Change-Id: I4c5e4ba7dab4079b3a45dfca62afc61ee11af22c

Merge "POST updates to include new URI and response status in CoAP header"

POST updates to include new URI and response status in CoAP header

This was previously submitted as review # 330 but ran into issues during rebase. Abandoning 330 and creating this one. Review comments in 330 have been addressed.

Change-Id: I3f39718975421a9e8e22e953623a81b9c7aac623

Added code for C++ implementation on Delete on both server and client ends.
Patch2 : Incorporated review comments
Change-Id: If666017890ec38890934bb01e53c9f89d49fee42

Merge "Delete old testapp files. They do not work and are obsolete."

Delete old testapp files. They do not work and are obsolete.

Change-Id: I1b4c1148acef98885b25a42ea3281aa126181948

Made small change to sample app to ensure that all clients are notified
when a server's resource is deleted.

Change-Id: I7eae7c365d9b570ac377d18e50c72e0477862d08

Client requests to delete a resource.
*Patch 1:
  1. Process request at C apps' level.
  2. Move logic into the C stack instead of app level.
  3. C stack responds to request to a 'to be deleted' resource.
*Patch 2: Added fixes per Sachin's code review.
*Patch 3: Added improvements per Jesse's code review.
***Abandoned changeset #293
***Restarted with changeset #304
*Patch 1: Rebased and changed Makefile per Jesse's feedback.
*Patch 2: Removed 'mark to be deleted' logic per design changes
          and added checks for entity handler result code.
*Patch 3: Fixed build error due to missing newly added coap code.
*Patch 4: Added improvements per Joey's feedbacks.
*Patch 5: Removed extra semicolon in enum to avoid build error.
*Patch 6: Rebased from Joey's changeset #313 and applied my changes on top.
*Patch 7: Rebased of master which has Joey's changeset #313, and added my changes on top.
*Patch 8: Added improvements per comments from Joey and Vijay

Change-Id: Iacec6f56c9f7ce7f61bfda8a47e84368edaf822e

Supporting QoS in C++ layer:
1) Modified the QoS enums so they are not transport specific
2) Overleaded member functions in Platform and Resource to allow specifying QoS
3) Modified internal function hierarchy to pass QoS to C layer
4) Added examples for C++: simpleclientHQ and simpleserverHQ to demo the usage of QoS
5) Modified examples to use the new QoS enums
6) Minor mods in the C layer
7) Minfor fix for OCICUC tests
8) Minfor fix for arduino sample
9) Fixes for received reviews
10)Rebase on to master
11)OCICUC : Platform bare pointer to reference

Change-Id: I5789935e16c851e7d47921a9acdfb15a363daa8e

Merge "Added logic to notify all observers when a resource has been deleted by OCDeleteResource API or when OCStop() is called. This will effectively tell all observers that "this" resource is no longer available. This information can be used by a client to infer that the server is going down."

Merge "Mods while working User Story US1742"

Added logic to notify all observers when a resource has been deleted by
OCDeleteResource API or when OCStop() is called. This will effectively tell
all observers that "this" resource is no longer available. This information
can be used by a client to infer that the server is going down.

If any more changes are req'q to encompass the new "Client able to delete
a resource" user story, a new changeset will be generated.

To come: A changeset containing a set (server and client) of examples
using this functionality.

Patch 2: Thuyen rebased on top of mine and for some reason needed to change
the author of my changeset, but doesn't appear to have made any changes to
my changeset other than rebasing it onto master.

Patch 3: Rebased on to newest master and made changes req' by Sachin and
Sudarshan in Patch 1 comments.

Change-Id: I343a5589c2988c45da95ba6d15685c21fa4704b6

Mods while working User Story US1742

1. Add additional stack and heap debug info to Arduino ocserver.cpp
2. Add optional debug info to ocmalloc.c.  Disabled by default.
3. Minor parameter name change to libcoap/mem.h
4. Mod libcoap/uthash.h to use OCMalloc instead of malloc

Rebased in response to broken build

Change-Id: I17e443f8dd1c0035140a1c0778532354e68900d8

Fix OCDoHandle comparation error

OCDoHanlde is 'void *', its size is uncertain. Compare the pointer
directly or must know its real size.

Change-Id: I91614fae4cee52637e9e0aeb42bfd150b63f34ee

Revert "Replaced listenCallback PTree with cereal"

This reverts commit 7e5b9d431a6ddf13c537c0e10bde0b7ecd866f19.

Change-Id: I3ccf81e13f0491394de58da7de4f7d64c439bbeb

Merge branch 'master' of ssh://oic/oic-resource

Replaced listenCallback PTree with cereal

Change-Id: I3ded4d42c495c73d5f2fb1d564722758ba8e5879

Merge "Fix but in OCUtility.cpp, exception information string wasn't being included."

Fix but in OCUtility.cpp, exception information string wasn't being included.

Change-Id: I89a20e6411220eb3be4604a8b8c12e5a6621c644

Merge "Fix type conflict"

Merge "Example to demonstrate the idea of sending json object as string in OCRepresenation"

Fix type conflict

In address.h, coap_address_t is defined as OCDevAddr, it should be used
as coap_address_t not 'struct coap_address_t'

Change-Id: I110ebc2e7064dad6588d4b3ae7625a2edde58093

Added clean up logic in makefile for ocicuc to remove new app
"small_example."

Change-Id: I564c7468582f449127ae54e96298cb7733b7ee6e

Example to demonstrate the idea of sending json object as string in OCRepresenation

Change-Id: I73a7c35241a55da856174223e121c3d808ed476f

Merge "fix for arduinodue "install" target for ocserver sample application."

Merge "Made additions to buildSCript and various makefiles to ensure a better "make clean" of all targets in the entire repo."

Merge "Tidy up logging call sites; throw in a few places. Remove old code (from my change). Clip lines. (ouch!)"

Made additions to buildSCript and various makefiles to ensure a better
"make clean" of all targets in the entire repo.

Patch 2: Cleaned up error that caused loop when "clean" target was called
by buildScipt.mk or makefile at root of repo.

Change-Id: Ib3b93a0a3a8b26c226ffdbf4851fa7176392a9b4

Vector of OCRepresentations type in OCRepresentation

Change-Id: If1097afc87010d7d930e125d83b140e85bad11ef

fix for arduinodue "install" target for ocserver sample application.

Change-Id: If49a841f5a4a65016bf6493cbf734f6a3d1a51dd

Tidy up logging call sites; throw in a few places.
Remove old code (from my change).
Clip lines. (ouch!)

Change-Id: If83a68d24b70e8f576d73334eace7366c98a57a4

Merge "Revert "Fix to remove quotes around array of strings as attribute value. Patch 2: Fixed same for array of bool Change-Id: Ib732ac8aff643cc579f57aafee27dee39b2f33f7""

Revert "Fix to remove quotes around array of strings as attribute value. Patch 2: Fixed same for array of bool Change-Id: Ib732ac8aff643cc579f57aafee27dee39b2f33f7"

This reverts commit f9f016f533d200725ad39f1af34b67af529a8949.

Change-Id: Idbc9ddb9b93e189f3fb55c1cb0ede0d0a4989303

Merge "1. Fix for IOT-58. Set the resourceInterfaceName to be default when it is set to empty string. NULL is still considered to be an error condition. 2. NULL string replaced by default interface using strlen(). 3. Removed extra tab 4. Removed unnecessary paranthesis. Changed !strlen() to strlen()==0 5. Changed default interface name macro. Change-Id: I91c159e880f9e5c905c47415b48577f623ff3311"

Merge "Fix compile error"

1. Fix for IOT-58. Set the resourceInterfaceName to be default when it is set to empty string.
NULL is still considered to be an error condition.
2. NULL string replaced by default interface using strlen().
3. Removed extra tab
4. Removed unnecessary paranthesis. Changed !strlen() to strlen()==0
5. Changed default interface name macro.
Change-Id: I91c159e880f9e5c905c47415b48577f623ff3311

Fix compile error

const modifier is required for new avr-gcc compiler

Change-Id: Ic6edb878e6de720c115ac30f88ad8ce3c2618e53

Merge "Broke out build script into variability.. Added support to build either in debug or release. It defaults to release. Added support to build either ethernet or wifi for both Mega and Due. Converged build process of sample app for Arduino Due into the sample app makefile for Arduino Mega."

1. Added a check in ocstack.c for the Maximum JSON payload that can be sent.
2. Added specific error message to result. Changes in if() as suggested.
4. Replaced tab with spaces and moved the check before any processing begins.
Change-Id: Id06ce2f7deaf643acd4f3cf4fcdd9ce30e2eddbc

Merge "Updated example program."

1. Changes made to Sample application (ocserver) in csdk. No need to
hardcode the JSON strings anymore. Added createJSONResponse() function
for this purpose. Edited the makefile to include the cJSON library.
2. Made some changes to the makefile.
3. Jenkins Failure
4. Indentation Changes
5. 6 . 7.  8. 9.  Gerrit issues
11. Rebase
Change-Id: Ice9dd8cd614bc78cb2ea034ec15ea1a61db1770a

Build system base on scons(oic-resource)

Currently support:
 1) build linux binary on ubuntu
 2) build android binary on windows/ubuntu/Mac OSX
 3) build darwin binary on Mac OSX
 4) build IOS on MAC OSX
 5) build arduino on linux

CPU ARCHs support:
 1) linux: host default arch, support cross compile for other arch
 2) android: x86, x86_64, armeabi, armeabi-v7a, armeabi-v7a-hard,
arm64-v8a
 3) darwin: i386, x86_64
 4) IOS: i386/x86_64 (Simulator), armv7, armv7s, arm64
 5) arduino: avr, arm

Last test and synced with commit 55e34d19a791cf01655012d1daef3ebd18e0e9f4

Change-Id: I40df2989cf48e0f32767c0aebdcc734aa0362e15

Updated example program.

Change-Id: I02d9a652a1b80ec7eebe0dde1039dae7b5dbdbec

Merge "Fix to remove quotes around array of strings as attribute value. Patch 2: Fixed same for array of bool Change-Id: Ib732ac8aff643cc579f57aafee27dee39b2f33f7"

Fix to remove quotes around array of strings as attribute value.
Patch 2:
Fixed same for array of bool
Change-Id: Ib732ac8aff643cc579f57aafee27dee39b2f33f7

Changes in C++ for vendor specific header options to be sent from client to server and vice-versa
Patch 2:
Code clean up
Fixes for sending header options from client to server
Patch 3:
Added additional comments;
Addressed review comments, added more comments.
Added implementation for sending header options from server to client.
Updated related sample files.
Added comments
Rebased with master
Completed additional changes for building.
Patch 4:
Removed unnecessary statements in makefile.

Change-Id: Ie984051615632263a0f653dc5a1347f8b50dae10

Broke out build script into variability.. Added support to build either in
debug or release. It defaults to release. Added support to build either
ethernet or wifi for both Mega and Due. Converged build process of sample
app for Arduino Due into the sample app makefile for Arduino Mega.

Patch 2: Fixed ocserver sample application.Made change suggested by Sachin.
Also edited README from typo.

Change-Id: Ib4d76f255be37e970899562e6e5f2ebbb246fd06

Fixing returned error codes in OCProcessPresence.

Change-Id: I1976a240bdd440708acfc4e10bfc0bc06437c35a

Changed std::mutex to std::recursive_mutex in the client/server wrappers.  This will allow us to call create/get/put/etc from the entity handler\!

Added the simpleclient/simpleserver changes from Sashi's review.  Should be able to use them to test post.

Change-Id: I3e96e0a1915e4da0e4ce5880db0b00c6dd73f247

Make exception and error handling more consistent through the top-level API.

Following up on Sudarshan's comments.

Change-Id: I8e19a89952f5235abbab43f1eb266604b5a4f1ac

Add README to ocicuc; add small_example near-minimal demo program;
Miscellaneous small fixes.

Change-Id: I8fc65b8eca5c05e231c2820ccdb8ca8237eb084e

Merge "1.Optimization to reduce number of strlen() calls. 2. Removed the extra blank line. 3. '?' is no longer considered a part of the query Change-Id: Ieabc1352a4cccc504ae1caac59b154aa7a9c0997"

1.Optimization to reduce number of strlen() calls.
2. Removed the extra blank line.
3. '?' is no longer considered a part of the query
Change-Id: Ieabc1352a4cccc504ae1caac59b154aa7a9c0997

Merge "Added file describing artifact locations from all build processes. Right now, it only describes artifact locations that the buildScript.mk file outputs after being run. However, this will be updated to include all arduino due targets, as well the additional arduino mega wifi target."

Merge "Arduino Due needs to use DWORD to read address of strings stored in PROGMEM"

Added file describing artifact locations from all build processes. Right
now, it only describes artifact locations that the buildScript.mk file
outputs after being run. However, this will be updated to include
all arduino due targets, as well the additional arduino mega wifi target.

Patch 2: Added note in README to refer to artifact_output_locations.txt
file.

Change-Id: I7a7f5ed639c27457bb495891723ca0fa344e2adb

Merge "Tech writer reviewed edits for programmer's guide"

Arduino Due needs to use DWORD to read address of strings stored in PROGMEM

Change-Id: Ie01b61df3fecfa12f64d0acfdfdf6f1bd5f8f581

Merge "Added mechanism to set URI in the regular entity handler's request. This allows app's entity handler to retrieve the URI of resource from a request."

Tech writer reviewed edits for programmer's guide

Change-Id: Ia8703d6017105b819395edfba378521848e56ea1
Signed-off-by: shamit patel <shamit.patel@intel.com>

Cancel threads before exiting in
csdk/stack/samples/linux/SimpleClientServer/ocserver.cpp and ocservercoll.cpp.
If threads are not canceled before exiting program, valgrind shows lost bytes.
While not really a stack problem, it could be distracting while trying to
use valgrind to find future memory leaks in the stack.

Change-Id: I653916211d6115381fae33cf95338c83b395d6fe

Added mechanism to set URI in the regular entity handler's request.
This allows app's entity handler to retrieve the URI of resource from a request.

Patch 2:
Addressed review comments - Added lock before map operation

Change-Id: Iaa5cad56580046bc05efc974d4a7e65213856bfd

Merge "IOT-57 fix.  We were mis-assigning a pointer during construction"

IOT-57 fix.  We were mis-assigning a pointer during construction

Change-Id: Ifc2301b6b7a9146940290c00d1fb13568e8472a9

Makefile changes req'd for a global build script. Also, global build
script implementation "buildScript.mk." Updated base README to reflect
the options and requirements for running this build script.

Patch 2: Edited makefiles so that developers CAN continue to use the build
processes they're used to. Also added 'all_dev' target that will build as
much of everything the build server can (including OCICUC) to prevet
regression in the build process as much as possible with a full-blown
build script.

Patch 3: Had to edit oc_logger makefile to support the new directory output
structure from the edits of this changeset. Also took care of issues
that Erich pointed out.

Patch 4: Added notes to README to state what is NOT being built by the
current iteration of buildScript.mk. Also edited lines of README that Doug
had mentioned needed updated in Patch 3.

Patch 5: Edited mistype in README and edited csdk/makefile as suggested by
Sachin.

Note: Arduino Due is not a part of the build script process.  This changeset
needs to be verified on the build server and put in place as the sole build
process. Once this is done, Support for building the Arduino Due will go in
place. Also, Ethernet Vs. Wifi build support will go in place at that time.
Right now everything is Ethernet.

Change-Id: Ie9938d9f3069aff1858141565dc66b4e2ece9e98

Fixed upsets in C Stack that caused breakage in the Arduino Build.

Change-Id: I6462963404db66d9c8af7543269d02d5591c718c

Release notes for Sept 30, 2014 release.
Patch 2:
Review comments addressed
Change-Id: Ifa00dee2d1e79d285a35fe750a92b9cba3ebe073

Merge "1 Bug fix to check for length of uri and query separately. 2 Addressed the code review feedback. No longer using strtok, moved check logic to separate function. 3 Optimized the code to use strlen only once. 4. Fixed Build failure"