Merge "Added style formatting config files for iotivity coding standards"

Merge "Fixed InProcServerWrapper to be thread safe"

Merge "Fixed simpleclient crash caused by curResource being altered"

Fixed InProcServerWrapper to be thread safe

A few of the variables/collections in the server wrapper were not
properly protected against multi-threading.  This fix adds a mutex
to the class to ensure that all accessed items are protected.
Additionally, they are moved out of the root namespace to prevent
polution.

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

Merge "Fix failed unit tests for CSDK stack."

Merge "C++ Avoid crash in case of Invalid/Empty Payload"

Fix failed unit tests for CSDK stack.

The existing unit tests in stacktests.cpp had not kept up with
ongoing modifications to the C API.  Modified failing tests to
align with latest C API functionality.

Also modified to remove limitation that required the eth0 network
interface to be available on the test machine.  If eth0 was not
available, some tests would fail.

Also added a local function to handle C stack initialization and
modified tests to call this function.  Added local functions to
intialize the number of expected resources and resource index
depending on whether presence is defined.

Re-grouped related test cases and created common group test names.

Change-Id: Ief59854d762ef9837f8b1d5b95bebc6514bd545a
Signed-off-by: Doug Hudson <douglas.hudson@intel.com>

Change the definition of GROUP_INTERFACE.

The definition of GROUP_INTERFACE as "oc.mi.c"
does not get big meaning.
So, we change the definition to "oc.mi.grp".
Regarding opinion of Caiwen.

Issue : [IOT-253]

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

Merge "Fixed invalid memory read, removed dead code"

Merge "Adding OK Response Code for POST"

Merge "Fixing repeated EH calls when client retransmits requests"

C++ Avoid crash in case of Invalid/Empty Payload

This change is based on a bug reported by SHP wherein
they want to be able to send response with empty payload.
Instead of throwing an error, we now send an empty
OCRepresentation() object to the client callback.

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

Added style formatting config files for iotivity coding standards

The style config files are used to apply IoTivity coding style, as
described on the wiki https://wiki.iotivity.org/iotivity_coding_standards.

Change-Id: Ifd3937bebde658a76aa7c9918b4c699c1384f341
Signed-off-by: William Dieter <william.r.dieter@intel.com>
Signed-off-by: Ossama Othman <ossama.othman@intel.com>

Removed executable bit from non-executable files.

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

Merge "Add quick guide into the readme"

[SSM] Fix issues about initializing SSM

1. modify to return error code when xml parsing error occurs
2. apply new SSM Interface
3. modify sample apps according to the new interface design
4. modify ResourceFinder to stop checking presence when SSM terminates

Change-Id: Id5cd6aade44fe531cb547400d15bb343ba28655a
Signed-off-by: Minji Park <minjii.park@samsung.com>

Merge changes I5acd9c57,I90e7a69e

* changes:
  [SSM] Fix context query parser issues
  [SSM] Add omitted arduino sample and soft sensor sample

[PPM] fix Plugin Manager Destructor segment fault.

1. Modify PluginManager Destructor.
2. Remove "free(handle)" and add "dlclose(handle)".

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

[SSM] Fix context query parser issues

This patch resolves issues that related to context query

which occures infinite loop and crash

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

[SSM] Add omitted arduino sample and soft sensor sample

1. Add omitted arduino sample app
2. Modify previous arduino sample to use proper resource name
3. Add IndoorTrajectory Soft Sensor

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

Adding OK Response Code for POST

Earlier, we were setting response code for POST only if a new resource
is created assuming that the value is by default OK.
TE tests showed error on using different version of Ubuntu.
Modified simpleserver to explicitly add OK status.
Added constructors to OCResourceRequest and
OCResourceResponse classes to avoid undefined behavior.
This Fixes IOT-128.

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

Fixed simpleclient crash caused by curResource being altered

The foundResource function is called asynchronously, and was written
in a way that was not thread safe. This fix adds a lock and exit logic
so that it works prooperly and only uses the first found curResource
that matches its requirements.

NOTE: This change (author: Erich Keane) is from 01.org.

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

Merge "Add C++ Unit tests"

Fixing repeated EH calls when client retransmits requests

(Bug IOT-239) When a client makes a request to a slow resource
and it does not receive an ACK for that request, it retransmits
the requests which resulted in calling the server's entity handler
several times and multilpe slow responses. When a server
detects a repeated request it should not process it. Also, when
the client receives a slow response it should stop retransmitting
requests.

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

Fixed invalid memory read, removed dead code

Debug.c's coap_print_addr had its functionality completely removed
in the past.  However, this resulted in returning SOMETHING (not zero!)
despite not actually correctly writing to the addr parameter.

Since this parameter is uninitialized when sent to this function,
and the return value is used to determine whether it correctly added
info to the addr, the debug print following this (net.c:972) attempted
to read the uninitialized data, potentially causing memory corruption.

This fix removes the 'dead' code, then returns a zero (failure, nothing
written).

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

Merge "Fixing OCCancel with High_QOS in observe isn't deleting CB."

Fixing OCCancel with High_QOS in observe isn't deleting CB.

(Bug IOT-126) Calling OCCancel should result in deleting all
associated structures including callbacks. This also should
happen when we deregister observation.

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

Add C++ Unit tests

Added unit tests for OCPlatform and OCResource C++ APIs.
All tests are passing

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

Merge "Removing getting started and how to guides from API references."

Removing getting started and how to guides from API references.

Developer content will be available on iotivity.org going forward, so
there is no need for these files in the API docs code. Direct links were
added to the Home.txt for the iotivity.org/documentation and
wiki.iotivity.org sites.

IOT-218 - Removed the "M1 release" text in title from Doxygen output
IOT-219 - Remove "About OIC" , "Programmer's guide" and "how to"

Change-Id: I28e26328a82cb6ff037ce451a5ed6fee368ad635
Signed-off-by: mshaver <michael.r.shaver@intel.com>

Merge "Fix some issues of GroupSynchronization  - findGroup() with null callback parameter issue is fixed.    : return OC_STACK_ERROR, when callback parameter is passed with a null value.  - joinGroup() failure when there is no group match with group type parameter is fixed.  - deleteGroup() failure when delete group with no child is fixed."

Add quick guide into the readme

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

Fix some issues of GroupSynchronization
- findGroup() with null callback parameter issue is fixed.
  : return OC_STACK_ERROR, when callback parameter is passed with a null value.
- joinGroup() failure when there is no group match with group type parameter is fixed.
- deleteGroup() failure when delete group with no child is fixed.

Erase "else" statement for guard condition in findCallback NULL checking.

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

Add missing dependency to readme.

Change-Id: I86cf5280edbe70dfa5c8f0076c9475ee18e5ab0b
Signed-off-by: Sanjeev BA <as2902.b@samsung.com>

Merge "Fix some issues of GroupManager"

fix "Coding convention followed in this commit"

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

Fix some issues of GroupManager

    - subscribeCollectionPresence()
        - IOT-197 : Handle null parameter
        - IOT-198 : Check that resource has child resource.
    - findCandidateResources()
        - IOT-210 : Handle null parameter

Change-Id: I6961aa776e5883890da226f07d3cfcf64571279c
Signed-off-by: Jack Lee <jjack.lee@samsung.com>

Merge "This a complete version of OICMiddle. The ReST capability has limited functionality, which we hope to expand shortly after release.  We believe the other parts of OICMiddle are sufficiently robust that the application will be useful as both demo and example in the OIC 0.9.0 release."

This a complete version of OICMiddle.
The ReST capability has limited functionality, which we hope to expand
shortly after release.  We believe the other parts of OICMiddle are
sufficiently robust that the application will be useful as both demo
and example in the OIC 0.9.0 release.

In a partial review prior to moving from 01.org to IoTivity, one
reviewer provided 50 comments.  All of the comments were addressed,
though not all were agreed with.  In some cases, comments were added
to the code to address the concern.  Others may require further
discussion.

Change-Id: I3e52de09a0737dd8dd0323fa2a20e4c94224f9c7
Signed-off-by: John.Light <john.j.light@intel.com>
Signed-off-by: Sudarshan Prasad <sudarshan.prasad@intel.com>

Merge "Updated the README files to reference the master build in root"

Updated the README files to reference the master build in root

Change-Id: Ibeb95e7a872ca51e29656a825c37247e48d89fe5
Signed-off-by: Patrick Lankswert <patrick.lankswert@intel.com>

Merge "Android SDK - Initial commit and build system updates"

Android SDK - Initial commit and build system updates

This commit add makes the auto_build process more complete by
adding the following:

1) Auto build now always performs the dependency check
2) Auto build now always attempts to build the Android stack artifacts.
3) Transitioned the Android build process to the v4.9 tool chain
4) Android build now depends on the internal boost dependencies
5) Add an Android compatibility layer for the code that is using
   the C++ 11 basic string functions with is not supported in the
   Android toolchain yet.
6) Added static libraries to many of the shared libraries for the
   Android SDK

NOTE:
a) This commit conflicts with the work that Caiwen was making in the
   C++11 basic_string function compatibility. I will close with him.
b) Expat is needed by the protocol plug in manager for Android.
   This still needs to be closed by release. The PPM build for
   Android under SCONS is currently disabled.

This is the Android SDK client API.
Validation has not been completed. However, this is all new
functionality and does not have any negative impact on any
existing features. Until validation is completed this
functionality should be considered experimental.

This code based currently uses gradle to build the SDK.
It does require that the Android libraries for IoTivity
be built previously using the appropriate SCONS scripts.

This additional patch attempts to transition the boost
build to using git instead of a download in hopes that
the build machine will be happier about it.

Final patch makes Android build optional in auto_build

Another patch to add support for 64-bit hosts

Another patch set to reduce parallel builds to 4

Another patch to add licensing header to two files that we missing
in gerrit review #72

Change-Id: I9695ebf3ea271ec28b69a4fcec509a6c2b7c46a4
Signed-off-by: Patrick Lankswert <patrick.lankswert@intel.com>

Merge "Added build dependencies script"

Merge "OICSensorBoard: App for Intel Edison Platform."

Added build dependencies script

Created build script that prepares two dependencies (cereal and
boost) for use by the build environment. This also adds the
dependency builder in the auto build script used by the continuous
integation servers.

Change-Id: I776a3b3f208b1ea0c253cc693d639c19b603fbca
Signed-off-by: Patrick Lankswert <patrick.lankswert@intel.com>

Merge "Add spec file for Tizen build"

Merge "Fix soft-sensor-manager service build errors"

Merge "Fix android build compatibility problems"

Merge "Added services programmer's guides without DRM for June Young"

Added services programmer's guides without DRM for June Young

Change-Id: Ifa7b359cabae84d3dcd1b48feac3fe7b4423af69
Signed-off-by: George Bowden <george.bowden@intel.com>

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

OICSensorBoard: App for Intel Edison Platform.

(Please refer to https://oic-review.01.org/gerrit/#/c/668/8/
for prior review status)

Build for OICSensorBoard is NOT included in the
standard Iotivity build.

It requires the Yocto cross-toolchain for
a chosen target, and would therefore be
built separately.

It demonstrates Iotivity Server capabilities through
the integration of an add-on breadboard that hosts
temperature, ambient light and LED sensors.
In addition, there is a command-line Iotivity Client
application to test server functionality from a
separate machine.

The included scons build script expects to build
the client for Linux (tested on Ubuntu 12.04)
and the server to be built via the Yocto
cross-toolchain for Edison.

Brief instructions for constructing this
toolchain are provided in the included README.

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

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 android build compatibility problems

1. Some C++11 features are used in IoTivity. Old Android NDK may doesn't
support a part of these features directly. Implement and include it if
it is required.

2. Fix libc function missing problem. Some functions, e.g. rand, srand.
strtof ... are static inline prior to android-L(it is defined and
implemented in the header file). So before android-L libc.so doesn't
include them. If build on android-L and run on an old platform(earlier
than android-L), there will be 'can't locate xxx' problem.

e.g.:
rand, it is defined in stdlib.h as "static __inline__ int rand() { ....} in
the ndk < 10. it's defined as 'extern int rand();' from ndk-10x. With old
ndk, if a file (x.c) used 'rand' function, when build x.c, 'rand' is compiled
into x.o. 'rand' won't exit in neither libc.so nor libc.a. If build an
application with ndk-10x and run it on emulator or device(developed with
NDK < 10), it requires implement 'rand' function by owerself or there
will be be 'can't locate rand' problem.

For convenience, add these functions in a file, if necessary, build it as
library and auto link into binary.

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

Fix soft-sensor-manager service build errors

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

Add spec file for Tizen build

Change-Id: I9d70f69b174c1fedad12116c2dba75f84773e50b
Signed-off-by: Arron Wang <arron.wang@intel.com>

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"