- code format update in mms_server_common.c and additional NULL check after memory allocation

src/mms/iso_mms/server/mms_server_common.c

@@ -44,21 +44,23 @@ mmsServer_write_out(const void *buffer, size_t size, void *app_key)
 MmsPdu_t*
 mmsServer_createConfirmedResponse(uint32_t invokeId)
 {
-	MmsPdu_t* mmsPdu = (MmsPdu_t*) GLOBAL_CALLOC(1, sizeof(MmsPdu_t));
+    MmsPdu_t* mmsPdu = (MmsPdu_t*)GLOBAL_CALLOC(1, sizeof(MmsPdu_t));
 
-	mmsPdu->present = MmsPdu_PR_confirmedResponsePdu;
+    if (mmsPdu)
+    {
+        mmsPdu->present = MmsPdu_PR_confirmedResponsePdu;
 
-	asn_long2INTEGER(&(mmsPdu->choice.confirmedResponsePdu.invokeID),
+        asn_long2INTEGER(&(mmsPdu->choice.confirmedResponsePdu.invokeID), invokeId);
-		invokeId);
+    }
 
-	return mmsPdu;
+    return mmsPdu;
 }
 
 static void
 mapErrorTypeToErrorClass(MmsError errorType, uint8_t* tag, uint8_t* value)
 {
-    switch (errorType) {
+    switch (errorType)
-
+    {
     case MMS_ERROR_ACCESS_OBJECT_ACCESS_UNSUPPORTED:
         *tag = 0x87; /* access */
         *value = 1;
@@ -361,47 +363,44 @@ exit_function:
 int
 mmsServer_getLowIndex(AlternateAccess_t* alternateAccess)
 {
-	if (alternateAccess->list.array[0]->choice.unnamed->choice.selectAccess.present
+    if (alternateAccess->list.array[0]->choice.unnamed->choice.selectAccess.present ==
-				== AlternateAccessSelection__selectAccess_PR_index)
+        AlternateAccessSelection__selectAccess_PR_index)
-	{
+    {
-		long index;
+        long index;
-		asn_INTEGER2long(
+        asn_INTEGER2long(&alternateAccess->list.array[0]->choice.unnamed->choice.selectAccess.choice.index, &index);
-				&alternateAccess->list.array[0]->choice.unnamed->choice.selectAccess.choice.index,
+
-				&index);
+        return (int)index;
-
+    }
-		return (int) index;
+
-	}
+    if (alternateAccess->list.array[0]->choice.unnamed->choice.selectAccess.present ==
-
+        AlternateAccessSelection__selectAccess_PR_indexRange)
-	if (alternateAccess->list.array[0]->choice.unnamed->choice.selectAccess.present
+    {
-				== AlternateAccessSelection__selectAccess_PR_indexRange)
+        long index;
-	{
+        asn_INTEGER2long(
-		long index;
+            &alternateAccess->list.array[0]->choice.unnamed->choice.selectAccess.choice.indexRange.lowIndex, &index);
-		asn_INTEGER2long(
+
-				&alternateAccess->list.array[0]->choice.unnamed->choice.selectAccess.choice.indexRange.lowIndex,
+        return (int)index;
-				&index);
+    }
-
+
-		return (int) index;
+    return -1;
-	}
-
-	return -1;
 }
 
 int
 mmsServer_getNumberOfElements(AlternateAccess_t* alternateAccess)
 {
-	if (alternateAccess->list.array[0]->choice.unnamed->choice.selectAccess.present
+    if (alternateAccess->list.array[0]->choice.unnamed->choice.selectAccess.present ==
-				== AlternateAccessSelection__selectAccess_PR_indexRange)
+        AlternateAccessSelection__selectAccess_PR_indexRange)
-	{
+    {
-		long number;
+        long number;
 
-		asn_INTEGER2long(
+        asn_INTEGER2long(
-				&alternateAccess->list.array[0]->choice.unnamed->choice.selectAccess.choice.indexRange.numberOfElements,
+            &alternateAccess->list.array[0]->choice.unnamed->choice.selectAccess.choice.indexRange.numberOfElements,
-				&number);
+            &number);
 
-		return (int) number;
+        return (int)number;
-	}
+    }
 
-	return 0;
+    return 0;
 }
 
 MmsNamedVariableList
@@ -430,23 +429,23 @@ mmsServer_getNamedVariableListWithName(LinkedList namedVariableLists, const char
 void
 mmsServer_deleteVariableList(LinkedList namedVariableLists, char* variableListName)
 {
-	LinkedList previousElement = namedVariableLists;
+    LinkedList previousElement = namedVariableLists;
-	LinkedList element = LinkedList_getNext(namedVariableLists);
+    LinkedList element = LinkedList_getNext(namedVariableLists);
 
-	while (element)
+    while (element)
     {
-		MmsNamedVariableList varList = (MmsNamedVariableList) element->data;
+        MmsNamedVariableList varList = (MmsNamedVariableList)element->data;
 
-		if (strcmp(MmsNamedVariableList_getName(varList), variableListName)	== 0)
+        if (strcmp(MmsNamedVariableList_getName(varList), variableListName) == 0)
         {
-			previousElement->next = element->next;
+            previousElement->next = element->next;
-			GLOBAL_FREEMEM(element);
+            GLOBAL_FREEMEM(element);
-			MmsNamedVariableList_destroy(varList);
+            MmsNamedVariableList_destroy(varList);
 
-			break;
+            break;
-		}
+        }
 
-		previousElement = element;
+        previousElement = element;
-		element = LinkedList_getNext(element);
+        element = LinkedList_getNext(element);
-	}
+    }
 }