csdk: Declare void functions as strict prototype 99/23699/2
authorPhilippe Coval <philippe.coval@osg.samsung.com>
Mon, 13 Nov 2017 23:39:06 +0000 (00:39 +0100)
committerAshok Babu Channa <ashok.channa@samsung.com>
Wed, 13 Dec 2017 06:45:32 +0000 (06:45 +0000)
If built along -Wstrict-prototypes it will raise and error,
this is the case on TizenRT.

More patches to come to enable this flag by default.

Bug: https://jira.iotivity.org/browse/IOT-2539
Change-Id: I15ef08db79fd3e775dd063adf46410813268c238
Signed-off-by: Philippe Coval <philippe.coval@osg.samsung.com>
16 files changed:
resource/csdk/connectivity/api/cautilinterface.h
resource/csdk/connectivity/inc/caadapterutils.h
resource/csdk/connectivity/inc/catcpadapter.h
resource/csdk/connectivity/inc/catcpinterface.h
resource/csdk/connectivity/src/tcp_adapter/arduino/catcpserver_eth.cpp
resource/csdk/connectivity/src/tcp_adapter/catcpadapter.c
resource/csdk/connectivity/src/tcp_adapter/catcpserver.c
resource/csdk/connectivity/util/inc/caconnectionmanager.h
resource/csdk/connectivity/util/src/camanager/caconnectionmanager.c
resource/csdk/connectivity/util/src/camanager/camessagearbiter.c
resource/csdk/connectivity/util/src/camanager/camessagearbiter.h
resource/csdk/connectivity/util/src/camanager/capolicymanager.c
resource/csdk/connectivity/util/src/camanager/capolicymanager.h
resource/csdk/stack/include/internal/occonnectionmanager.h
resource/csdk/stack/src/occonnectionmanager.c
resource/csdk/stack/src/oickeepalive.c

index e98c02f..101da02 100644 (file)
@@ -138,13 +138,13 @@ uint16_t CAGetAssignedPortNumber(CATransportAdapter_t adapter, CATransportFlags_
  * Initializes the Connection Manager
  * @return ::CA_STATUS_OK or ERROR CODES (::CAResult_t error codes in cacommon.h).
  */
-CAResult_t CAUtilCMInitailize();
+CAResult_t CAUtilCMInitailize(void);
 
 /**
  * Terminate the Connection Manager
  * @return ::CA_STATUS_OK or ERROR CODES (::CAResult_t error codes in cacommon.h).
  */
-CAResult_t CAUtilCMTerminate();
+CAResult_t CAUtilCMTerminate(void);
 
 /**
  * Update RemoteDevice Information for Connection Manager
@@ -158,7 +158,7 @@ CAResult_t CAUtilCMUpdateRemoteDeviceInfo(const CAEndpoint_t *endpoint, bool isC
  * Reset RemoteDevice Info. for Connection Manager
  * @return ::CA_STATUS_OK or Appropriate error code.
  */
-CAResult_t CAUtilCMResetRemoteDeviceInfo();
+CAResult_t CAUtilCMResetRemoteDeviceInfo(void);
 
 /**
  * Set Connection Manager configuration
@@ -244,7 +244,7 @@ CAResult_t CAUtilSetLEScanInterval(jint intervalTime, jint workingCount);
  * stop LE scan.
  * @return  ::CA_STATUS_OK or ::CA_NOT_SUPPORTED
  */
-CAResult_t CAUtilStopLEScan();
+CAResult_t CAUtilStopLEScan(void);
 #endif //__JAVA__
 
 // BLE util
index 4fd7ad0..aa883a1 100644 (file)
@@ -234,7 +234,7 @@ void CANativeJNISetJavaVM(JavaVM *jvm);
  * Called from adapters to get JavaVM object.
  * @return  JVM object.
  */
-JavaVM *CANativeJNIGetJavaVM();
+JavaVM *CANativeJNIGetJavaVM(void);
 
 /**
  * get method ID for method Name and class
@@ -259,7 +259,7 @@ bool CACheckJNIException(JNIEnv *env);
  * To Delete other Global References
  * Called during CATerminate to remove global references
  */
-void CADeleteGlobalReferences();
+void CADeleteGlobalReferences(JNIEnv *env);
 
 #ifdef __ANDROID__
 /**
index c70a08a..753fff0 100644 (file)
@@ -98,7 +98,7 @@ CAResult_t CAInitializeTCP(CARegisterConnectivityCallback registerCallback,
  * Start TCP Interface adapter.
  * @return  ::CA_STATUS_OK or Appropriate error code.
  */
-CAResult_t CAStartTCP();
+CAResult_t CAStartTCP(void);
 
 /**
  * Start listening server for receiving connect requests.
@@ -107,7 +107,7 @@ CAResult_t CAStartTCP();
  * number and as per OIC Specification.
  * @return  ::CA_STATUS_OK or Appropriate error code.
  */
-CAResult_t CAStartTCPListeningServer();
+CAResult_t CAStartTCPListeningServer(void);
 
 /**
  * Stops listening server from receiving connect requests.
@@ -116,7 +116,7 @@ CAResult_t CAStartTCPListeningServer();
  * number and as per OIC Specification.
  * @return  ::CA_STATUS_OK or Appropriate error code.
  */
-CAResult_t CAStopTCPListeningServer();
+CAResult_t CAStopTCPListeningServer(void);
 
 /**
  * Start discovery servers for receiving advertisements.
@@ -125,7 +125,7 @@ CAResult_t CAStopTCPListeningServer();
  * number as per OIC Specification.
  * @return  ::CA_STATUS_OK or Appropriate error code.
  */
-CAResult_t CAStartTCPDiscoveryServer();
+CAResult_t CAStartTCPDiscoveryServer(void);
 
 /**
  * Sends data to the endpoint using the adapter connectivity.
@@ -169,19 +169,19 @@ CAResult_t CAGetTCPInterfaceInformation(CAEndpoint_t **info, size_t *size);
  * Read Synchronous API callback.
  * @return  ::CA_STATUS_OK or Appropriate error code.
  */
-CAResult_t CAReadTCPData();
+CAResult_t CAReadTCPData(void);
 
 /**
  * Stops Unicast, servers and close the sockets.
  * @return  ::CA_STATUS_OK or Appropriate error code.
  */
-CAResult_t CAStopTCP();
+CAResult_t CAStopTCP(void);
 
 /**
  * Terminate the TCP connectivity adapter.
  * Configuration information will be deleted from further use.
  */
-void CATerminateTCP();
+void CATerminateTCP(void);
 
 /**
  * Set connection status changes callback to process KeepAlive.
index 468f41e..53d854c 100644 (file)
@@ -133,7 +133,7 @@ CAResult_t CATCPStartServer(const ca_thread_pool_t threadPool);
 /**
  * Stop TCP server.
  */
-void CATCPStopServer();
+void CATCPStopServer(void);
 
 /**
  * Set this callback for receiving data packets from peer devices.
@@ -185,7 +185,7 @@ CAResult_t CADisconnectTCPSession(CATCPSessionInfo_t *removedData);
 /**
  * Disconnect all connection from TCP Server.
  */
-void CATCPDisconnectAll();
+void CATCPDisconnectAll(void);
 
 /**
  * Get TCP connection information from list.
index a601b60..d393dc2 100644 (file)
@@ -134,7 +134,7 @@ CAResult_t CATCPStartServer()
     return CA_STATUS_OK;
 }
 
-void CATCPStopServer()
+void CATCPStopServer(void)
 {
     OIC_LOG(DEBUG, TAG, "IN");
     caglobals.tcp.terminate = true;
index 39a15e5..5799f32 100644 (file)
@@ -102,9 +102,9 @@ static void CATCPPacketReceivedCB(const CASecureEndpoint_t *sep,
 static void CATCPErrorHandler(const CAEndpoint_t *endpoint, const void *data,
                               size_t dataLength, CAResult_t result);
 
-static CAResult_t CATCPInitializeQueueHandles();
+static CAResult_t CATCPInitializeQueueHandles(void);
 
-static void CATCPDeinitializeQueueHandles();
+static void CATCPDeinitializeQueueHandles(void);
 
 static void CATCPSendDataThread(void *threadData);
 
@@ -118,7 +118,7 @@ static void CADataDestroyer(void *data, uint32_t size);
 static int32_t CAQueueTCPData(bool isMulticast, const CAEndpoint_t *endpoint,
                              const void *data, size_t dataLength, bool encryptedData);
 
-CAResult_t CATCPInitializeQueueHandles()
+CAResult_t CATCPInitializeQueueHandles(void)
 {
     // Check if the message queue is already initialized
     if (g_sendQueueHandle)
@@ -148,7 +148,7 @@ CAResult_t CATCPInitializeQueueHandles()
     return CA_STATUS_OK;
 }
 
-void CATCPDeinitializeQueueHandles()
+void CATCPDeinitializeQueueHandles(void)
 {
     CAQueueingThreadDestroy(g_sendQueueHandle);
     OICFree(g_sendQueueHandle);
@@ -297,7 +297,7 @@ void CATCPAdapterHandler(CATransportAdapter_t adapter, CANetworkStatus_t status)
     }
 }
 
-static void CAInitializeTCPGlobals()
+static void CAInitializeTCPGlobals(void)
 {
     caglobals.tcp.ipv4.fd = OC_INVALID_SOCKET;
     caglobals.tcp.ipv4s.fd = OC_INVALID_SOCKET;
index 86061c9..0cdfd57 100644 (file)
@@ -118,10 +118,10 @@ static CATCPConnectionHandleCallback g_connectionCallback = NULL;
  */
 static CATCPSessionInfo_t *g_sessionList = NULL;
 
-static CAResult_t CATCPCreateMutex();
-static void CATCPDestroyMutex();
-static CAResult_t CATCPCreateCond();
-static void CATCPDestroyCond();
+static CAResult_t CATCPCreateMutex(void);
+static void CATCPDestroyMutex(void);
+static CAResult_t CATCPCreateCond(void);
+static void CATCPDestroyCond(void);
 static CASocketFd_t CACreateAcceptSocket(int family, CASocket_t *sock);
 static void CAAcceptConnection(CATransportFlags_t flag, CASocket_t *sock);
 static void CAFindReadyMessage();
@@ -160,7 +160,7 @@ do \
         FD_SET(caglobals.tcp.TYPE.fd, FDS); \
     }
 
-static void CATCPDestroyMutex()
+static void CATCPDestroyMutex(void)
 {
     if (g_mutexObjectList)
     {
@@ -169,7 +169,7 @@ static void CATCPDestroyMutex()
     }
 }
 
-static CAResult_t CATCPCreateMutex()
+static CAResult_t CATCPCreateMutex(void)
 {
     if (!g_mutexObjectList)
     {
@@ -184,7 +184,7 @@ static CAResult_t CATCPCreateMutex()
     return CA_STATUS_OK;
 }
 
-static void CATCPDestroyCond()
+static void CATCPDestroyCond(void)
 {
     if (g_condObjectList)
     {
@@ -193,7 +193,7 @@ static void CATCPDestroyCond()
     }
 }
 
-static CAResult_t CATCPCreateCond()
+static CAResult_t CATCPCreateCond(void)
 {
     if (!g_condObjectList)
     {
@@ -1135,7 +1135,7 @@ CAResult_t CATCPStartServer(const ca_thread_pool_t threadPool)
     return CA_STATUS_OK;
 }
 
-void CATCPStopServer()
+void CATCPStopServer(void)
 {
     if (caglobals.tcp.terminate)
     {
@@ -1495,7 +1495,7 @@ CAResult_t CADisconnectTCPSession(CATCPSessionInfo_t *removedData)
     return CA_STATUS_OK;
 }
 
-void CATCPDisconnectAll()
+void CATCPDisconnectAll(void)
 {
     oc_mutex_lock(g_mutexObjectList);
     CATCPSessionInfo_t *session = NULL;
index afc2cba..e209444 100644 (file)
@@ -47,13 +47,13 @@ extern "C"
  * Initializes the connection manager context.
  * @return ::CA_STATUS_OK or ERROR CODES (::CAResult_t error codes in cacommon.h).
  */
-CAResult_t CACMInitialize();
+CAResult_t CACMInitialize(void);
 
 /**
  * Terminate the connection manager context.
  * @return ::CA_STATUS_OK or ERROR CODES (::CAResult_t error codes in cacommon.h).
  */
-CAResult_t CACMTerminate();
+CAResult_t CACMTerminate(void);
 
 /**
  * Get request/response message to send.
@@ -88,7 +88,7 @@ CAResult_t CACMUpdateRemoteDeviceInfo(const CAEndpoint_t *endpoint, bool isCloud
  * Reset Remote Device Info.
  * @return ::CA_STATUS_OK or Appropriate error code.
  */
-CAResult_t CACMResetRemoteDeviceInfo();
+CAResult_t CACMResetRemoteDeviceInfo(void);
 
 /**
  * Set callback to handle connection status
index 1cff6af..eb7d539 100644 (file)
 #define TAG "OIC_CM"
 
 
-CAResult_t CACMInitialize()
+CAResult_t CACMInitialize(void)
 {
     OIC_LOG(DEBUG, TAG, "CACMInitialize");
     return CAMsgArbiterInitialize();
 }
 
-CAResult_t CACMTerminate()
+CAResult_t CACMTerminate(void)
 {
     OIC_LOG(DEBUG, TAG, "CACMTerminate");
 
@@ -80,7 +80,7 @@ CAResult_t CACMUpdateRemoteDeviceInfo(const CAEndpoint_t *endpoint, bool isCloud
     return CAMsgArbiterUpdateDeviceInfo(endpoint, isCloud);
 }
 
-CAResult_t CACMResetRemoteDeviceInfo()
+CAResult_t CACMResetRemoteDeviceInfo(void)
 {
     return CAMsgArbiterResetDeviceInfo();
 }
index 929e2dd..a849a33 100644 (file)
@@ -56,21 +56,21 @@ static oc_mutex g_remoteDeviceInfoListMutex = NULL;
  * Initializes all required mutex variable for messge arbiter.
  * @return ::CA_STATUS_OK or ERROR CODES (::CAResult_t error codes in cacommon.h).
  */
-static CAResult_t CAMsgArbiterInitMutexVariables();
+static CAResult_t CAMsgArbiterInitMutexVariables(void);
 
 /**
  * Terminates all required mutex variable for messge arbiter.
  */
-static void CAMsgArbiterTerminateMutexVariables();
+static void CAMsgArbiterTerminateMutexVariables(void);
 
-CAResult_t CAMsgArbiterInitialize()
+CAResult_t CAMsgArbiterInitialize(void)
 {
     OIC_LOG(DEBUG, TAG, "CAMsgArbiterInitialize");
 
     return CAMsgArbiterInitMutexVariables();
 }
 
-CAResult_t CAMsgArbiterTerminate()
+CAResult_t CAMsgArbiterTerminate(void)
 {
     OIC_LOG(DEBUG, TAG, "CAMsgArbiterTerminate");
 
@@ -86,7 +86,7 @@ CAResult_t CAMsgArbiterTerminate()
     return CA_STATUS_OK;
 }
 
-static CAResult_t CAMsgArbiterInitMutexVariables()
+static CAResult_t CAMsgArbiterInitMutexVariables(void)
 {
     if (NULL == g_remoteDeviceInfoListMutex)
     {
@@ -100,7 +100,7 @@ static CAResult_t CAMsgArbiterInitMutexVariables()
     return CA_STATUS_OK;
 }
 
-static void CAMsgArbiterTerminateMutexVariables()
+static void CAMsgArbiterTerminateMutexVariables(void)
 {
     if (g_remoteDeviceInfoListMutex)
     {
@@ -259,7 +259,7 @@ CAResult_t CAMsgArbiterUpdateDeviceInfo(const CAEndpoint_t *endpoint, bool isClo
     return ret;
 }
 
-CAResult_t CAMsgArbiterResetDeviceInfo()
+CAResult_t CAMsgArbiterResetDeviceInfo(void)
 {
     OIC_LOG(DEBUG, TAG, "IN");
 
index c27f308..4273b10 100644 (file)
@@ -38,12 +38,12 @@ extern "C"
  * Initializes the Message Arbiter
  * @return ::CA_STATUS_OK or ERROR CODES (::CAResult_t error codes in cacommon.h).
  */
-CAResult_t CAMsgArbiterInitialize();
+CAResult_t CAMsgArbiterInitialize(void);
 
 /**
  * Terminate the Message Arbiter
  */
-CAResult_t CAMsgArbiterTerminate();
+CAResult_t CAMsgArbiterTerminate(void);
 
 /**
  * Update Device Info. in managed data
@@ -57,7 +57,7 @@ CAResult_t CAMsgArbiterUpdateDeviceInfo(const CAEndpoint_t *endpoint, bool isClo
  * Reset Device Info in managed data
  * @return ::CA_STATUS_OK or ERROR CODES (::CAResult_t error codes in cacommon.h).
  */
-CAResult_t CAMsgArbiterResetDeviceInfo();
+CAResult_t CAMsgArbiterResetDeviceInfo(void);
 
 /**
  * Get request/response message to send.
index 842788b..4350f31 100644 (file)
@@ -93,7 +93,7 @@ CAResult_t CAPolicyMgrGetConfigure(CAConnectUserPref_t *connPrefer)
     return CA_STATUS_OK;
 }
 
-void CAPolicyMgrTerminate()
+void CAPolicyMgrTerminate(void)
 {
     if (g_userConfigureMutex)
     {
index 0e1b3c9..5a6d96c 100644 (file)
@@ -58,7 +58,7 @@ CAResult_t CAPolicyMgrGetConfigure(CAConnectUserPref_t *connPrefer);
 /**
  * terminate mutex
  */
-void CAPolicyMgrTerminate();
+void CAPolicyMgrTerminate(void);
 
 #ifdef __cplusplus
 } /* extern "C" */
index 5fbc73e..092abd7 100644 (file)
  *
  * @return ::OC_STACK_OK or Appropriate error code.
  */
-OCStackResult OCCMInitialize();
+OCStackResult OCCMInitialize(void);
 
 /**
  * Terminate the Connection Manager
  *
  * @return ::OC_STACK_OK or Appropriate error code.
  */
-OCStackResult OCCMTerminate();
+OCStackResult OCCMTerminate(void);
 
 /**
  *  Discover Resources by Connection Manager
index c548016..5effd50 100644 (file)
@@ -88,7 +88,7 @@ static bool OCCMHasDeviceId(const char *uri);
  *
  * @return ::OC_STACK_OK or ERROR CODES (::OCStackResult_t error codes in octypes.h).
  */
-static OCStackResult OCCMFindResource();
+static OCStackResult OCCMFindResource(void);
 
 /**
  * Callback function for OCCMFindResource.
@@ -278,7 +278,7 @@ static void OCConnectionStateChangedHandler(const CAEndpoint_t *info, bool isCon
     }
 }
 
-static OCStackResult OCCMFindResource()
+static OCStackResult OCCMFindResource(void)
 {
     OIC_LOG_V(INFO, TAG, "%s", __func__);
 
index 34eb073..17d226f 100644 (file)
@@ -142,13 +142,13 @@ static OCStackApplicationResult PingRequestCallback(void* ctx, OCDoHandle handle
  * This function creates KeepAlive resource.
  * @return  ::OC_STACK_OK or Appropriate error code.
  */
-static OCStackResult CreateKeepAliveResource();
+static OCStackResult CreateKeepAliveResource(void);
 
 /**
  * This function deletes KeepAlive resource.
  * @return  ::OC_STACK_OK or Appropriate error code.
  */
-static OCStackResult DeleteKeepAliveResource();
+static OCStackResult DeleteKeepAliveResource(void);
 
 /**
  * API to handle the GET request received for a KeepAlive resource.
@@ -306,7 +306,7 @@ OCStackResult TerminateKeepAlive(OCMode mode)
     return OC_STACK_OK;
 }
 
-OCStackResult CreateKeepAliveResource()
+OCStackResult CreateKeepAliveResource(void)
 {
     OIC_LOG(DEBUG, TAG, "InitKeepAliveResource IN");
 
@@ -333,7 +333,7 @@ OCStackResult CreateKeepAliveResource()
     return result;
 }
 
-OCStackResult DeleteKeepAliveResource()
+OCStackResult DeleteKeepAliveResource(void)
 {
     OIC_LOG(DEBUG, TAG, "DeleteKeepAliveResource IN");