#include "credresource.h"
#include "policyengine.h"
#include "srmutility.h"
-#include "amsmgr.h"
#include "oic_string.h"
#include "oic_malloc.h"
#include "securevirtualresourcetypes.h"
static CAResponseCallback gResponseHandler = NULL;
//Error Callback handler
static CAErrorCallback gErrorHandler = NULL;
-//Persistent Storage callback handler for open/read/write/close/unlink
-static OCPersistentStorage *gPersistentStorageHandler = NULL;
//Provisioning response callback
static SPResponseCallback gSPResponseHandler = NULL;
/**
- * A single global Policy Engine context will suffice as long
+ * A single global Request context will suffice as long
* as SRM is single-threaded.
*/
-PEContext_t g_policyEngineContext;
+SRMRequestContext_t g_requestContext;
/**
* Function to register provisoning API's response callback.
gSPResponseHandler = respHandler;
}
-void SetResourceRequestType(PEContext_t *context, const char *resourceUri)
+void SetRequestedResourceType(SRMRequestContext_t *context)
{
- context->resourceType = GetSvrTypeFromUri(resourceUri);
+ context->resourceType = GetSvrTypeFromUri(context->resourceUri);
}
-static void SRMSendUnAuthorizedAccessresponse(PEContext_t *context)
+// Send the response (context->responseInfo) to the requester
+// (context->endPoint).
+static void SRMSendResponse(SRMRequestContext_t *context)
{
- CAResponseInfo_t responseInfo = {.result = CA_EMPTY};
-
- if (NULL == context ||
- NULL == context->amsMgrContext->requestInfo)
+ if (NULL != context
+ && NULL != context->requestInfo
+ && NULL != context->endPoint)
{
- OIC_LOG_V(ERROR, TAG, "%s : NULL Parameter(s)",__func__);
- return;
- }
-
- memcpy(&responseInfo.info, &(context->amsMgrContext->requestInfo->info),
- sizeof(responseInfo.info));
- responseInfo.info.payload = NULL;
- responseInfo.result = CA_UNAUTHORIZED_REQ;
- responseInfo.info.dataType = CA_RESPONSE_DATA;
- if (CA_STATUS_OK == CASendResponse(context->amsMgrContext->endpoint, &responseInfo))
- {
- OIC_LOG(DEBUG, TAG, "Succeed in sending response to a unauthorized request!");
+ if (CA_STATUS_OK == CASendResponse(context->endPoint,
+ &(context->responseInfo)))
+ {
+ OIC_LOG_V(DEBUG, TAG, "SRM response sent.");
+ context->responseSent = true;
+ }
+ else
+ {
+ OIC_LOG_V(ERROR, TAG, "SRM response failed.");
+ }
}
else
{
- OIC_LOG(ERROR, TAG, "Failed in sending response to a unauthorized request!");
+ OIC_LOG_V(ERROR, TAG, "%s : NULL Parameter(s)",__func__);
}
+
+ return;
}
-void SRMSendResponse(SRMAccessResponse_t responseVal)
+// Based on the context->responseVal, either call the entity handler for the
+// request (which must send the response), or send an ACCESS_DENIED response.
+void SRMGenerateResponse(SRMRequestContext_t *context)
{
- OIC_LOG(DEBUG, TAG, "Sending response to remote device");
+ OIC_LOG_V(INFO, TAG, "%s : entering function.", __func__);
- if (IsAccessGranted(responseVal) && gRequestHandler)
+ // If Access Granted, validate parameters and then pass request
+ // on to resource endpoint.
+ if (IsAccessGranted(context->responseVal))
{
- OIC_LOG_V(INFO, TAG, "%s : Access granted. Passing Request to RI layer", __func__);
- if (!g_policyEngineContext.amsMgrContext->endpoint ||
- !g_policyEngineContext.amsMgrContext->requestInfo)
+ if(NULL != gRequestHandler
+ && NULL != context->endPoint
+ && NULL != context->requestInfo)
+ {
+ OIC_LOG_V(INFO, TAG, "%s : Access granted, passing req to endpoint.",
+ __func__);
+ gRequestHandler(context->endPoint, context->requestInfo);
+ context->responseSent = true; // SRM counts on the endpoint to send
+ // a response.
+ }
+ else // error condition; log relevant msg then send DENIED response
{
- OIC_LOG_V(ERROR, TAG, "%s : Invalid arguments", __func__);
- SRMSendUnAuthorizedAccessresponse(&g_policyEngineContext);
- goto exit;
+ OIC_LOG_V(ERROR, TAG, "%s : Null values in context.", __func__);
+ context->responseVal = ACCESS_DENIED_POLICY_ENGINE_ERROR;
+ context->responseInfo.result = CA_INTERNAL_SERVER_ERROR;
+ SRMSendResponse(context);
}
- gRequestHandler(g_policyEngineContext.amsMgrContext->endpoint,
- g_policyEngineContext.amsMgrContext->requestInfo);
}
- else
+ else // Access Denied
{
- OIC_LOG_V(INFO, TAG, "%s : ACCESS_DENIED.", __func__);
- SRMSendUnAuthorizedAccessresponse(&g_policyEngineContext);
+ OIC_LOG_V(INFO, TAG, "%s : Access Denied; sending CA_UNAUTHORIZED_REQ.",
+ __func__);
+ // TODO: in future version, differentiate between types of DENIED.
+ // See JIRA issue 1796 (https://jira.iotivity.org/browse/IOT-1796)
+ context->responseInfo.result = CA_UNAUTHORIZED_REQ;
+ SRMSendResponse(context);
}
-
-exit:
- //Resetting PE state to AWAITING_REQUEST
- SetPolicyEngineState(&g_policyEngineContext, AWAITING_REQUEST);
+ return;
}
-/**
- * Handle the request from the SRM.
- *
- * @param endPoint object from which the response is received.
- * @param requestInfo contains information for the request.
- */
-void SRMRequestHandler(const CAEndpoint_t *endPoint, const CARequestInfo_t *requestInfo)
+// Set the value of context->resourceUri, based on the context->requestInfo.
+void SetResourceUriAndType(SRMRequestContext_t *context)
{
- OIC_LOG(DEBUG, TAG, "Received request from remote device");
-
- bool isRequestOverSecureChannel = false;
- if (!endPoint || !requestInfo)
- {
- OIC_LOG(ERROR, TAG, "Invalid arguments");
- return;
- }
-
- // Copy the subjectID
- OicUuid_t subjectId = {.id = {0}};
- OicUuid_t nullSubjectId = {.id = {0}};
- memcpy(subjectId.id, requestInfo->info.identity.id, sizeof(subjectId.id));
-
- // if subject id is null that means request is sent thru coap.
- if (memcmp(subjectId.id, nullSubjectId.id, sizeof(subjectId.id)) != 0)
- {
- OIC_LOG(INFO, TAG, "request over secure channel");
- isRequestOverSecureChannel = true;
- }
+ char *uri = strstr(context->requestInfo->info.resourceUri, "?");
+ size_t position = 0;
- //Check the URI has the query and skip it before checking the permission
- char *uri = strstr(requestInfo->info.resourceUri, "?");
- int position = 0;
if (uri)
{
//Skip query and pass the resource uri
- position = uri - requestInfo->info.resourceUri;
+ position = uri - context->requestInfo->info.resourceUri;
}
else
{
- position = strlen(requestInfo->info.resourceUri);
+ position = strlen(context->requestInfo->info.resourceUri);
}
if (MAX_URI_LENGTH < position || 0 > position)
{
- OIC_LOG(ERROR, TAG, "Incorrect URI length");
+ OIC_LOG_V(ERROR, TAG, "Incorrect URI length.");
return;
}
- SRMAccessResponse_t response = ACCESS_DENIED;
- char newUri[MAX_URI_LENGTH + 1];
- OICStrcpyPartial(newUri, MAX_URI_LENGTH + 1, requestInfo->info.resourceUri, position);
+ OICStrcpyPartial(context->resourceUri, MAX_URI_LENGTH + 1,
+ context->requestInfo->info.resourceUri, position);
- SetResourceRequestType(&g_policyEngineContext, newUri);
+ // Set the resource type.
+ context->resourceType = GetSvrTypeFromUri(context->resourceUri);
- // Form a 'Error', 'slow response' or 'access deny' response and send to peer
- CAResponseInfo_t responseInfo = {.result = CA_EMPTY};
- memcpy(&responseInfo.info, &(requestInfo->info), sizeof(responseInfo.info));
- responseInfo.info.payload = NULL;
- responseInfo.info.dataType = CA_RESPONSE_DATA;
+ return;
+}
- OCResource *resPtr = FindResourceByUri(newUri);
- if (NULL != resPtr)
+// Check if this request is asking to access a "sec" = true resource
+// over an unsecure channel. This type of request is forbidden with
+// the exception of a few SVRs (see Security Specification).
+void CheckRequestForSecResourceOverUnsecureChannel(SRMRequestContext_t *context)
+{
+ // if request is over unsecure channel, check resource type
+ if(false == context->secureChannel)
{
- // All vertical secure resources and SVR resources other than DOXM & PSTAT should reject request
- // over coap.
- if ((((resPtr->resourceProperties) & OC_SECURE)
- && (g_policyEngineContext.resourceType == NOT_A_SVR_RESOURCE))
- || ((g_policyEngineContext.resourceType < OIC_SEC_SVR_TYPE_COUNT)
- && (g_policyEngineContext.resourceType != OIC_R_DOXM_TYPE)
- && (g_policyEngineContext.resourceType != OIC_R_PSTAT_TYPE)))
+ OCResource *resPtr = FindResourceByUri(context->resourceUri);
+ if (NULL != resPtr)
{
- // if resource is secure and request is over insecure channel
- if (!isRequestOverSecureChannel)
+ // All vertical secure resources and SVR resources other than
+ // DOXM & PSTAT should reject requests over unsecure channel.
+ if ((((resPtr->resourceProperties) & OC_SECURE)
+ && (context->resourceType == NOT_A_SVR_RESOURCE))
+ || ((context->resourceType < OIC_SEC_SVR_TYPE_COUNT)
+ && (context->resourceType != OIC_R_DOXM_TYPE)
+ && (context->resourceType != OIC_R_PSTAT_TYPE)))
{
// Reject all the requests over coap for secure resource.
- responseInfo.result = CA_FORBIDDEN_REQ;
- if (CA_STATUS_OK != CASendResponse(endPoint, &responseInfo))
- {
- OIC_LOG(ERROR, TAG, "Failed in sending response to a unauthorized request!");
- }
- return;
+ context->responseVal = ACCESS_DENIED_SEC_RESOURCE_OVER_UNSECURE_CHANNEL;
+ context->responseInfo.result = CA_FORBIDDEN_REQ;
+ SRMSendResponse(context);
}
}
}
-#ifdef _ENABLE_MULTIPLE_OWNER_
- /*
- * In case of ACL and CRED, The payload required to verify the payload.
- * Payload information will be used for subowner's permission verification.
- */
- g_policyEngineContext.payload = (uint8_t*)requestInfo->info.payload;
- g_policyEngineContext.payloadSize = requestInfo->info.payloadSize;
-#endif //_ENABLE_MULTIPLE_OWNER_
- //New request are only processed if the policy engine state is AWAITING_REQUEST.
- if (AWAITING_REQUEST == g_policyEngineContext.state)
+ return;
+}
+
+void ClearRequestContext(SRMRequestContext_t *context)
+{
+ if (NULL == context)
{
- OIC_LOG_V(DEBUG, TAG, "Processing request with uri, %s for method, %d",
- requestInfo->info.resourceUri, requestInfo->method);
- response = CheckPermission(&g_policyEngineContext, &subjectId, newUri,
- GetPermissionFromCAMethod_t(requestInfo->method));
+
+ OIC_LOG(ERROR, TAG, "Null context.");
}
else
{
- OIC_LOG_V(INFO, TAG, "PE state %d. Ignoring request with uri, %s for method, %d",
- g_policyEngineContext.state, requestInfo->info.resourceUri, requestInfo->method);
+ // Clear context variables.
+ context->endPoint = NULL;
+ context->resourceType = OIC_RESOURCE_TYPE_ERROR;
+ memset(&context->resourceUri, 0, sizeof(context->resourceUri));
+ context->requestedPermission = PERMISSION_ERROR;
+ memset(&context->responseInfo, 0, sizeof(context->responseInfo));
+ context->responseSent = false;
+ context->responseVal = ACCESS_DENIED_POLICY_ENGINE_ERROR;
+ context->requestInfo = NULL;
+ context->secureChannel = false;
+ context->slowResponseSent = false;
+ context->subjectIdType = SUBJECT_ID_TYPE_ERROR;
+ memset(&context->subjectUuid, 0, sizeof(context->subjectUuid));
+#ifdef MULTIPLE_OWNER
+ memset(&context->payload, 0, context->payloadSize); // TODO Samsung reviewer: please confirm
+ context->payloadSize = 0; // TODO Samsung reviewer: please confirm
+#endif //MULTIPLE_OWNER
}
- if (IsAccessGranted(response) && gRequestHandler)
+ return;
+}
+
+// Returns true iff Request arrived over secure channel
+bool isRequestOverSecureChannel(SRMRequestContext_t *context)
+{
+ OicUuid_t nullSubjectId = {.id = {0}};
+
+ // if flag set, return true
+ if(context->endPoint->flags & CA_SECURE)
{
- gRequestHandler(endPoint, requestInfo);
- return;
+ return true;
+ }
+ // a null subject ID indicates CoAP, so if non-null, also return true
+ else if(memcmp(context->requestInfo->info.identity.id,
+ nullSubjectId.id, sizeof(context->requestInfo->info.identity.id)) != 0)
+ {
+ return true;
}
- VERIFY_NON_NULL(TAG, gRequestHandler, ERROR);
+ return false;
+}
- if (ACCESS_WAITING_FOR_AMS == response)
- {
- OIC_LOG(INFO, TAG, "Sending slow response");
+/**
+ * Entry point into SRM, called by lower layer to determine whether an incoming
+ * request should be GRANTED or DENIED.
+ *
+ * @param endPoint object from which the response is received.
+ * @param requestInfo contains information for the request.
+ */
+void SRMRequestHandler(const CAEndpoint_t *endPoint, const CARequestInfo_t *requestInfo)
+{
+ OIC_LOG(DEBUG, TAG, "Received request from remote device");
+
+ SRMRequestContext_t *ctx = &g_requestContext; // Always use our single ctx for now.
- UpdateAmsMgrContext(&g_policyEngineContext, endPoint, requestInfo);
- responseInfo.result = CA_EMPTY;
- responseInfo.info.type = CA_MSG_ACKNOWLEDGE;
+ ClearRequestContext(ctx);
+
+ if (!endPoint || !requestInfo)
+ {
+ OIC_LOG(ERROR, TAG, "Invalid endPoint or requestInfo; can't process.");
}
else
{
- /*
- * TODO Enhance this logic more to decide between
- * CA_UNAUTHORIZED_REQ or CA_FORBIDDEN_REQ depending
- * upon SRMAccessResponseReasonCode_t
- */
- OIC_LOG(INFO, TAG, "Sending for regular response");
- responseInfo.result = CA_UNAUTHORIZED_REQ;
+ ctx->endPoint = endPoint;
+ ctx->requestInfo = requestInfo;
+ ctx->requestedPermission = GetPermissionFromCAMethod_t(requestInfo->method);
+
+ // Copy the subjectID.
+ memcpy(ctx->subjectUuid.id,
+ requestInfo->info.identity.id, sizeof(ctx->subjectUuid.id));
+ ctx->subjectIdType = SUBJECT_ID_TYPE_UUID; // only supported type for now
+
+ // Set secure channel boolean.
+ ctx->secureChannel = isRequestOverSecureChannel(ctx);
+
+ // Set resource URI and type.
+ SetResourceUriAndType(ctx);
+
+ // Initialize responseInfo.
+ memcpy(&(ctx->responseInfo.info), &(requestInfo->info),
+ sizeof(ctx->responseInfo.info));
+ ctx->responseInfo.info.payload = NULL;
+ ctx->responseInfo.result = CA_INTERNAL_SERVER_ERROR;
+ ctx->responseInfo.info.dataType = CA_RESPONSE_DATA;
+
+ // Before consulting ACL, check if this is a forbidden request type.
+ CheckRequestForSecResourceOverUnsecureChannel(ctx);
+
+ // If DENIED response wasn't sent already, then it's time to check ACL.
+ if(false == ctx->responseSent)
+ {
+#ifdef MULTIPLE_OWNER // TODO Samsung: please verify that these two calls belong
+ // here inside this conditional statement.
+ // In case of ACL and CRED, The payload required to verify the payload.
+ // Payload information will be used for subowner's permission verification.
+ ctx->payload = (uint8_t*)requestInfo->info.payload;
+ ctx->payloadSize = requestInfo->info.payloadSize;
+#endif //MULTIPLE_OWNER
+
+ OIC_LOG_V(DEBUG, TAG, "Processing request with uri, %s for method %d",
+ ctx->requestInfo->info.resourceUri, ctx->requestInfo->method);
+ CheckPermission(ctx);
+ OIC_LOG_V(DEBUG, TAG, "Request for permission %d received responseVal %d.",
+ ctx->requestedPermission, ctx->responseVal);
+
+ // Now that we have determined the correct response and set responseVal,
+ // we generate and send the response to the requester.
+ SRMGenerateResponse(ctx);
+ }
}
- if (CA_STATUS_OK != CASendResponse(endPoint, &responseInfo))
+ if(false == ctx->responseSent)
{
- OIC_LOG(ERROR, TAG, "Failed in sending response to a unauthorized request!");
+ OIC_LOG(ERROR, TAG, "Exiting SRM without responding to requester!");
}
+
return;
-exit:
- responseInfo.result = CA_INTERNAL_SERVER_ERROR;
- if (CA_STATUS_OK != CASendResponse(endPoint, &responseInfo))
- {
- OIC_LOG(ERROR, TAG, "Failed in sending response to a unauthorized request!");
- }
}
/**
void SRMErrorHandler(const CAEndpoint_t *endPoint, const CAErrorInfo_t *errorInfo)
{
OIC_LOG_V(INFO, TAG, "Received error from remote device with result, %d for request uri, %s",
- errorInfo->result, errorInfo->info.resourceUri);
+ errorInfo->result, errorInfo->info.resourceUri);
if (gErrorHandler)
{
gErrorHandler(endPoint, errorInfo);
}
OCStackResult SRMRegisterHandler(CARequestCallback reqHandler,
- CAResponseCallback respHandler,
- CAErrorCallback errHandler)
+ CAResponseCallback respHandler, CAErrorCallback errHandler)
{
OIC_LOG(DEBUG, TAG, "SRMRegisterHandler !!");
if( !reqHandler || !respHandler || !errHandler)
OCStackResult SRMRegisterPersistentStorageHandler(OCPersistentStorage* persistentStorageHandler)
{
OIC_LOG(DEBUG, TAG, "SRMRegisterPersistentStorageHandler !!");
- if(!persistentStorageHandler)
- {
- OIC_LOG(ERROR, TAG, "The persistent storage handler is invalid");
- return OC_STACK_INVALID_PARAM;
- }
- gPersistentStorageHandler = persistentStorageHandler;
- return OC_STACK_OK;
+ return OCRegisterPersistentStorageHandler(persistentStorageHandler);
}
OCPersistentStorage* SRMGetPersistentStorageHandler()
{
- return gPersistentStorageHandler;
+ return OCGetPersistentStorageHandler();
}
OCStackResult SRMInitSecureResources()
DestroySecureResources();
}
-OCStackResult SRMInitPolicyEngine()
-{
- return InitPolicyEngine(&g_policyEngineContext);
-}
-
-void SRMDeInitPolicyEngine()
-{
- DeInitPolicyEngine(&g_policyEngineContext);
-}
-
bool SRMIsSecurityResourceURI(const char* uri)
{
if (!uri)