Remove Null Pointer Access in msg

Summary:
Some functions attempt to set the completion code of an ipmi message if
it's null. Remove those lines.

Reviewed By: zhdaniel12

Differential Revision: D37153996

fbshipit-source-id: a54e72ed9b5fb5b072ec80ead42942afb772f230

Author: GoldenBug

common/service/ipmi/oem_1s_handler.c

@@ -968,7 +968,6 @@ __weak void OEM_1S_READ_BIC_REGISTER(ipmi_msg *msg)
 	***********************************/
 	if (!msg) {
 		printf("pal_OEM_1S_READ_BIC_REGISTER: parameter msg is NULL\n");
-		msg->completion_code = CC_UNSPECIFIED_ERROR;
 		return;
 	}
 	if (msg->data_len != 5) {
@@ -996,7 +995,6 @@ __weak void OEM_1S_WRITE_BIC_REGISTER(ipmi_msg *msg)
 	***********************************/
 	if (!msg) {
 		printf("pal_OEM_1S_WRITE_BIC_REGISTER: parameter msg is NULL\n");
-		msg->completion_code = CC_UNSPECIFIED_ERROR;
 		return;
 	}
 	if (msg->data[4] < 1 || msg->data[4] > 4 || (msg->data_len != 5 + msg->data[4])) {