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

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"

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

Change-Id: I1d1c8ad39716a93dc32196e110d5a8f1df2ee959

Fix in makefile to build fridgeclient and fridgeserver samples

Change-Id: I1b42d33c6dc3b77e25579a6370b9bf8327edaa08

1. Supporting POST operation in C Stack.
2. Supporting POST operation in CPP Stack.
3. Updates to C sample application
4. Rebase
5. Fixed build issue.

Change-Id: I00b70a5eb8f4c1c911416d703e3c739b2a80ff7b

Merge "C++ programer's guide"

This commit to support exposing vendor specific options to application:

1) Define a structure defining the vendor specific options
2) Pass an array of vendor specific options needed with the call to OCDoResource
3) Vendor specific header options are checked against the range of acceptable vendor specific header options
4) Vendor specific header options are added to the pdu being sent
5) Vendor specific header options are parsed and added to OCEntityHandlerRequest and OCClientResponse structs
6) Rebased on to master
7) Adding a test case to occlient for sending vendor specific option
8) Adding The number of options to be sent and the ones received, also adding this infor to OCEntityHandlerRequest and OCClientResponse structs
9) Rebased on to master

Change-Id: I5d4c56028bf510fba57ad4fc440565fd4b87ca31

Merge "Added URI parameter for Device Entity Handler."

Merge "Modified makefiles so that each makefile could build its own dependencies or build only itself. Fixed .gitignore and makefiles base on review comments. Rebased as requested by Erich. Added fix to building with boost Cleaned up .gitignore"

Modified makefiles so that each makefile could build its own dependencies or build only itself.
Fixed .gitignore and makefiles base on review comments.
Rebased as requested by Erich.
Added fix to building with boost
Cleaned up .gitignore

Change-Id: I0829400e74694f2a7f8271ea72f93e7cd7be000c

Added URI parameter for Device Entity Handler.

Change-Id: If37e64f6578b6a8eccee9b9ea43690123d6ac2aa

APIs to setNULL and check for NULL in OCRepresenation

Change-Id: I3db71123c1362efaf7e641a845fa737057d56da5

Merge "1. Default device entity handler implementation in Cpp Stack. 2. Addressed review comments."

Merge "IOT-36 (Leak 2) : Fixing memory leaks Patch 2 : Taking care of some review comments. Patch 3 : Update in occoaphelper.c."

1. Default device entity handler implementation in Cpp Stack.
2. Addressed review comments.

Change-Id: I8745b604d62d9c2e49fce1cedb3af185ae2f5f5b

Merge "Fix cosmetic regression bug reported by Habteab and Mark. https://01.org/jira/browse/IOT-54"

Merge "Logging effort created a regression in InProcClientWrapper that caused get/observe to not work.  This fixes said regression"

Fix cosmetic regression bug reported by Habteab and Mark.
https://01.org/jira/browse/IOT-54

Change-Id: I6b7a946111800848249fea2258dff3253c7a5048

Merge "Simultaneous ACK and RST message received at Server causes SegFaults"

Merge "New APIs : hasAttribute, numberOfAttributes, erase in OCRepresentation."

Logging effort created a regression in InProcClientWrapper that caused get/observe to not work.  This fixes said regression

Change-Id: Id6ac04d3aec3ee1f9f0d93e2cfe06cc65324d1b5

IOT-36 (Leak 2) : Fixing memory leaks
Patch 2 : Taking care of some review comments.
Patch 3 : Update in occoaphelper.c.

Change-Id: I9bba4039e37cc94b0acad061a68d3f1cec427372

Simultaneous ACK and RST message received at Server causes SegFaults

Change-Id: I9f609c926ed80bba940a8ba5533cea755c181be6

New APIs : hasAttribute, numberOfAttributes, erase in OCRepresentation.

Change-Id: Ia6df616bdd73d7e73a34563896dd3c5d941b9a8b

c_logger implementation
Patch 2: rebase on master
Patch 3: adding license headers to missing files
Patch 4: rebase on master
Change-Id: I8c3d16000432df1de4b08633142ecc01ee423e0d

1.  Modify libcoap to use OCMalloc and OCFree.
2.  Modify uri.c to call coap_free instead of free.

Change-Id: If404da557bf3f2191ad1036b9a6ac91e10a96ca9

Merge "Remove hard-coded IP address from the samples. removed duplicate call to OCBuildIPv4Address OCInitCoAP."

Remove hard-coded IP address from the samples.
removed duplicate call to OCBuildIPv4Address OCInitCoAP.

Change-Id: I51ed3ff71997069e375043e37946180223a60141

Merge "IOT-36 : Fixing memory leaks"

Merge "1. Create csdk/ocmalloc directory 2. Create ocmalloc.c and ocmalloc.h 3. Remove OCMalloc and OCFree functions from ocstack.c and ocstack.h 4. Modify stack source modules to include ocmalloc.h 5. Modify stack makefiles to use ocmalloc.c and ocmalloc.h 6. Add doxygen function comments to ocmalloc.h 7. Remove #include from within #ifdef __cplusplus block 8. Add comment to ocmalloc.h describing the intention of the module."

Merge "Default device entity handler implementation in C Stack."

1. Create csdk/ocmalloc directory
2. Create ocmalloc.c and ocmalloc.h
3. Remove OCMalloc and OCFree functions from ocstack.c and ocstack.h
4. Modify stack source modules to include ocmalloc.h
5. Modify stack makefiles to use ocmalloc.c and ocmalloc.h
6. Add doxygen function comments to ocmalloc.h
7. Remove #include from within #ifdef __cplusplus block
8. Add comment to ocmalloc.h describing the intention of the module.

Change-Id: I453d4ced530d7539f8b00b22b3f651327ea6a42c

IOT-36 : Fixing memory leaks

Change-Id: I8ed2d398d79c43abf4d0131232c88f9ae8890f8f

Default device entity handler implementation in C Stack.

Patch 1-3: Implementation.

Patch 4-5: Review comments from Doug and Joey.

Patch 6: Joey -> Resolved 'defaultDeviceHandler' declaration
down to one instance.

Patch 7: Reverted Patch 6.

Patch 8: Removed the declaration from ocresource.c

Change-Id: Ic1a47f65ae6772ecf417fa5f7dcb384db029a4b7

Fridge example which shows a nice set of the std::bind functionality as well as how to use the ResourceObject creation.  Note that this will not function correctly without the csdk fix for resource commas.

Patch 2: Resolve Sudarshan's fixes
Patch 3: rebase & remove previous version of code
Change-Id: Ic7c5f3ae5da40cd8e6e82692e0f5e3b878f34489

Merge "Add const modifier"

Add const modifier

const modifier is required for new avr-gcc compiler

Change-Id: I1a7b4b4059632e0c6ea83a65d7514785575416cf

use of const reference where possible.

Change-Id: I4f47fdc385f58f2e52ce10bffa68ae0910e496c9

Merge "1. arrays in OCRepresentation 2. Fixed review comments from Jesse. 3. Refactored the code to get JSON from vector."

Merge "JSON Object implementation"

1. arrays in OCRepresentation
2. Fixed review comments from Jesse.
3. Refactored the code to get JSON from vector.

Change-Id: I1f31f8f0c242f53f56cf50f9d9e781d8ed770f8c

JSON Object implementation

1. Support for JSON object in OCRepresentation.
2. Code refactor for JSON composition in various parts of the stack into one place (OCRepresenation).
3. Similar code refactoring for JSON parsing needs to be done. I will revisit when we look into
   JSON parse library alternatives.
4. Implemented correct JSON composition. Earlier we were returning strings for all the json types.
   Now strings are only used for string types and JSON objects.
5. garageserver.cpp and garageclient.cpp examples to explain the user of JSON object with in
   the representation.

Change-Id: I9461db811c7b5e3b827d5704cde489aa2005e291

C++ programer's guide

Change-Id: Iab4d32ae3776cfbbe0f4c9947c71d11aaf605634

Non-discoverable resources cause bad JSON (trailing comma).  By putting the comma generation first, we buy ourselves some room

Change-Id: Ie270c22e0f61c657f15ecf45fcbdb96de72d495f

A fix for a bug reported by Omkar and Vijay:

The check for required uri lenght should consider that it includes the uri and query.

Change-Id: I1ca3e4709de02cb4f5217aba5133b8aa2bc4ee14

Makefile in ocicuc is not cleaning up all the object files.

Change-Id: Ia1d198b96b61a82c07c069b4545ad40460cf2f4c

Merge "Revert "Replace to_string and stoi""

Revert "Replace to_string and stoi"

We have decided that sprintf is not stable/future proof enough, so we are going to revert this change (and the other one) and investigate what it would take to make android-only implementations of tostring and stoi

This reverts commit dcfe2eb3bee0296aee3cc4453f4237c784251671.

Change-Id: I82bb37e42aa5b6809bdf6a6522dac678ba74aac0

Replace to_string and stoi

Part of C++11 features aren't supported by Android NDK, includes
to_string and stoi. Replace them with snprintf and atoi.

Change-Id: I6edda1856ab34c0e1dcc23daef0b105875a09189

Merge "Fixing Arduino compilation error"

Merge "Avoid redefine 'NDEBUG'"

Avoid redefine 'NDEBUG'

Change-Id: I1a8e1c5b0a8b7aa0c1ab19e292485e4e20ab90a5

Merge "Fixed warning (control reaching end of non void function) in OCPlatform.cpp"

Fixing Arduino compilation error

Change-Id: I030f3d66b54ab61bd866f434e3cadfee30ff4ac7

Fixed warning (control reaching end of non void function) in OCPlatform.cpp

Change-Id: Ibab82216537c0927794bfb57dbe9e97f52b4d028

Merge "Support for notifying specific observers using observation Id"

Added release notes for Sept 17th, 2014 release.

Change-Id: I3bf64deb84e54b2c9306c03688ff5d8be9fe4053

Support for notifying specific observers using observation Id

1. Initial code changes
2. Combine GET & observe registration calls to entity handler into a single call; support for CON messages in NotifySpecificObservers; updates sample server app
3. Added the support for immediate observe cancelation in OCCancel
4. Fixed the new interface to OCCancel in C++ layer
5. A temp solution for the case when observation sequence number wraps around.
6. Merging with Yamin's changes
7. Changed OCNotifyObservers to OCNotifyAllObservers
8. Build issue
9. Another Merge to support calling the entity handler when deregister an observer
10. Rebase & CPP Changes from Sudarshan and Sashi to support Notify specific observers.
11. Command line argument options for simpleserver (so it can choose between all observers or specific observers)
12. CPP bugfix.
13. Fixed review comments for CPP.
14. Changed COAP_DEFAULT_RESPONSE_TIMEOUT to 2 and all the sample applications to sleep accordingly.
15. Fixed review comment for new line.
16. Fixed review commend for line exceed.
17. Response to comments received on patch 11
18. Header file comments updated

Change-Id: I03aa7ab2a580899441b70bd2a76cca05575b192a

Removed <oic-resource>/csdk/android and <oic-resoruce>/csdk/controller plus <oic-resource>/csdk/libcoap-4.1.1/config.log and config.status

PatchSet 2: Erich Rebased for submission sake
Change-Id: I0d5a355924aca94847aa70f8adcb10c700164cdc
Signed-off-by: Charunethran PG <charunethran.panchalam.govindarajan@intel.com>

Merge "Partial Implementation of US1574:"

Partial Implementation of US1574:

OCRepresentation is now provides simpler interface to setValue and getValue from the applications.
Applications doesn't have to go through attributeMaps in OCRepresentation.
OCRepresentation now supports serialization/deserialization for primitive attributes (numbers, booleans and strings) so application doesn't have to deal with strings.

Change-Id: I09d323fc624d87c0b2a3c9b510935a6d5907a11b

Merge "Removed extraneous comment. The callback process is triggered from a lower point in the stack than from here. The consideration written in comment here has already been taken care of, as per Yamin."

Merge "IOT-48 : Updated makefiles for compiling OCStack with WiFi libraries for Arduino 1.0.5 Patch 2 : Included suggestions provided in Review comments."

Merge "Fixing a few warnings that we see coming out of the libcoap bits.h file."

Removed extraneous comment. The callback process is triggered from a
lower point in the stack than from here. The consideration written in
comment here has already been taken care of, as per Yamin.

Change-Id: I982922f8c87b42385e358cfe91b24bfee5ffc17f

Remove <ifaddrs.h>

This header file is unnecessary. It will break android build as
android ndk doesn't have this file.

Change-Id: Iacd08868d1e426fe73d6b4c31d0c67ce38df9e69

Removing references to OCReflect that were left behind but didnt cause build issues

Change-Id: I5480fa14d5ea4960b361d62a67563f45d1dbbfed

IOT-48 : Updated makefiles for compiling OCStack with WiFi libraries for Arduino 1.0.5
Patch 2 : Included suggestions provided in Review comments.

Change-Id: I70190ce85efcaf3b560687bff1ccace407778e44

Fixing a few warnings that we see coming out of the libcoap bits.h file.

Essentially, the result of the bit-shift on a uint8_t is always upconverted to a signed int despite it being impossible for it to be anything greater than 5 bits long (8 bits, shifted 3 more).  We cast to unsigned size_t to prevent the warning and to prevent unsigned vs signed comparison.

Change-Id: I47d05611e75e55240b55b800c9370dbc019aedd2

Patch 1: Sending RST in response to the receipt of a notification from a server
that is not a presence nor an observe notifcation AND there is not
a callback associated.

Patch 2: Rebase

Patch 3: Changed debug print out message.

Patch 4: Oops. I had too much message in my log message.

Change-Id: If8f949827e9accf0dc45b69a5568960578b388f5

Merge "Patch 1: Arduino pdu malloc fix."

Merge "Updated Arduino sample app to reflect the current SRAM available. \n This will help in debugging Arduino crash issues in low SRAM scenarios Patch 2 : Updated code to execute only for MEGA2560 boards."

Patch 1: Arduino pdu malloc fix.

Patch 2: Rebase

Patch 3: Changed implementation of fix to simply change size of
COAP_MAX_PDU_SIZE to 320 for Arduino only. This will stay the same
going forth, but there is an uncaught case that can only be fixed
in the next sprint(as there is not enough time)-- We must implement
blockwise transfer so that a PDU larger than 320 is properly
handled (i.e. broken up into chunks of 320 or smaller). A Rally task
has already been created for Sprint 8 for this.

Patch 4: Meant to use "else" instead of "elif".

Change-Id: I4df20d28c821f6fc2c98a6bf3672331264978be5

Updated Arduino sample app to reflect the current SRAM available. \n This will help in debugging Arduino crash issues in low SRAM scenarios
Patch 2 : Updated code to execute only for MEGA2560 boards.

Change-Id: If130815b1f8eedd9373e68e7e144ef8713a2f2cb

Merge "Revert "Doxygen run for *.h files""

Revert "Doxygen run for *.h files"

We should never have committed generated files.  When necessary, Shamit can generate these himself.

This reverts commit fdaa9cd9e05732b5087f11dcb40c2c66cc3a73e6.

Change-Id: Ibeb9f036ddf421d0d3e0f3d509ab6d2d92477db4

Remove unused variable

It's not used anywhere

Change-Id: I834b7e8f561e85dea45846d2bfe96ad90fed6ab8
Signed-off-by: Thiago Macieira <thiago.macieira@intel.com>

Fix mistaken use of && where & was intended

Found by Clang:
occoaphelper.c:93:59: warning: use of logical '&&' with constant operand [-Wconstant-logical-operand]

It's clearly meant to be a &, as seen by the right shift by 5 and the
constant of 31 (5 bits)

Change-Id: Ie35ad6c313ad53d766fc6fe88ffd03b462d49519
Signed-off-by: Thiago Macieira <thiago.macieira@intel.com>

Fix use of uninitialized variable

If ctx == NULL, then VERIFY_NON_NULL would goto exit, which would then
OCFree the uninitialized variables.

Found by Clang:

occoap.c:274:5: warning: variable 'rcvObserveOption' is used uninitialized whenever 'if' condition is true [-Wsometimes-uninitialized]
occoap.c:44:36: note: expanded from macro 'VERIFY_NON_NULL'
 #define VERIFY_NON_NULL(arg) { if (!arg) {OC_LOG_V(FATAL, TAG, "%s is NULL", #arg); goto exit;} }
                                    ^~~~
occoap.c:459:16: note: uninitialized use occurs here

--Erich: rebase on master

Change-Id: Ie829c6ac284dfd988eeeffad446d9872424e604a
Signed-off-by: Thiago Macieira <thiago.macieira@intel.com>

Fix warnings about diff and base possibly not initialized

This is a bogus warning by GCC, but it's easy to fix by having an "else"
case with no extra conditions.

Change-Id: I0f99527065138ff96c0613a57b0462613139900c
Signed-off-by: Thiago Macieira <thiago.macieira@intel.com>