aqhome apps: fixed taglist handling.

apps/aqhome-data/s_addvalue.c

@@ -31,6 +31,7 @@
 
 void AqHomeDataServer_HandleAddValue(AQH_OBJECT *o, AQH_OBJECT *ep, const AQH_MESSAGE *msg, const GWEN_TAG16_LIST *tagList)
 {
+  if (tagList) {
     AQHOME_SERVER *xo;
   
     xo=AqHomeDataServer_GetServerData(o);
@@ -60,6 +61,7 @@ void AqHomeDataServer_HandleAddValue(AQH_OBJECT *o, AQH_OBJECT *ep, const AQH_ME
       AQH_Endpoint_AddMsgOut(ep, outMsg);
     }
   }
+}
 
 
 

apps/aqhome-data/s_annvalue.c

@@ -31,6 +31,7 @@
 
 void AqHomeDataServer_HandleAnnounceValue(AQH_OBJECT *o, AQH_OBJECT *ep, const AQH_MESSAGE *msg, const GWEN_TAG16_LIST *tagList)
 {
+  if (tagList) {
     AQHOME_SERVER *xo;
 
     xo=AqHomeDataServer_GetServerData(o);
@@ -44,5 +45,6 @@ void AqHomeDataServer_HandleAnnounceValue(AQH_OBJECT *o, AQH_OBJECT *ep, const A
       }
     }
   }
+}
 
 

apps/aqhome-data/s_connect.c

@@ -31,6 +31,7 @@
 
 void AqHomeDataServer_HandleConnect(GWEN_UNUSED AQH_OBJECT *o, AQH_OBJECT *ep, const AQH_MESSAGE *msg, const GWEN_TAG16_LIST *tagList)
 {
+  if (tagList) {
     AQH_MESSAGE *outMsg;
     int resultCode=AQH_MSGDATA_RESULT_SUCCESS;
     char *clientId=NULL;
@@ -75,6 +76,7 @@ void AqHomeDataServer_HandleConnect(GWEN_UNUSED AQH_OBJECT *o, AQH_OBJECT *ep, c
                                     resultCode, NULL);
     AQH_Endpoint_AddMsgOut(ep, outMsg);
   }
+}
 
 
 

apps/aqhome-data/s_getdatapoints.c

@@ -62,6 +62,7 @@ static void _getAndSendLastDatapoint(AQH_STORAGE *storage, AQH_OBJECT *ep, const
 
 void AqHomeDataServer_HandleGetDataPoints(AQH_OBJECT *o, AQH_OBJECT *ep, const AQH_MESSAGE *recvdMsg, const GWEN_TAG16_LIST *tagList)
 {
+  if (tagList) {
     AQHOME_SERVER *xo;
   
     xo=AqHomeDataServer_GetServerData(o);
@@ -107,6 +108,7 @@ void AqHomeDataServer_HandleGetDataPoints(AQH_OBJECT *o, AQH_OBJECT *ep, const A
       }
     }
   }
+}
 
 
 

apps/aqhome-data/s_getdevices.c

@@ -49,6 +49,7 @@ static void _sendDeviceList(AQH_OBJECT *ep, const AQH_DEVICE_LIST *vl, uint32_t
 
 void AqHomeDataServer_HandleGetDevices(AQH_OBJECT *o, AQH_OBJECT *ep, const AQH_MESSAGE *msg, const GWEN_TAG16_LIST *tagList)
 {
+  if (tagList) {
     AQHOME_SERVER *xo;
   
     xo=AqHomeDataServer_GetServerData(o);
@@ -100,6 +101,7 @@ void AqHomeDataServer_HandleGetDevices(AQH_OBJECT *o, AQH_OBJECT *ep, const AQH_
       }
     }
   }
+}
 
 
 

apps/aqhome-data/s_getvalues.c

@@ -49,6 +49,7 @@ static void _sendValueList(AQH_OBJECT *ep, const AQH_VALUE_LIST *vl, uint32_t fl
 
 void AqHomeDataServer_HandleGetValues(AQH_OBJECT *o, AQH_OBJECT *ep, const AQH_MESSAGE *msg, const GWEN_TAG16_LIST *tagList)
 {
+  if (tagList) {
     AQHOME_SERVER *xo;
   
     xo=AqHomeDataServer_GetServerData(o);
@@ -101,6 +102,7 @@ void AqHomeDataServer_HandleGetValues(AQH_OBJECT *o, AQH_OBJECT *ep, const AQH_M
       }
     }
   }
+}
 
 
 

apps/aqhome-data/s_moddevice.c

@@ -36,6 +36,7 @@
 
 void AqHomeDataServer_HandleModDevice(AQH_OBJECT *o, AQH_OBJECT *ep, const AQH_MESSAGE *recvdMsg, const GWEN_TAG16_LIST *tagList)
 {
+  if (tagList) {
     AQHOME_SERVER *xo;
   
     xo=AqHomeDataServer_GetServerData(o);
@@ -98,6 +99,7 @@ void AqHomeDataServer_HandleModDevice(AQH_OBJECT *o, AQH_OBJECT *ep, const AQH_M
       AqHomeDataServer_SendResponseResultToEndpoint(ep, AQH_IpcMessage_GetMsgId(recvdMsg), resultCode);
     }
   }
+}
 
 
 

apps/aqhome-data/s_setdata.c

@@ -62,6 +62,7 @@ static void _subRqAbort(AQH_MSG_REQUEST *rq, int reason);
 
 void AqHomeDataServer_HandleSetData(AQH_OBJECT *o, AQH_OBJECT *epSrc, const AQH_MESSAGE *recvdMsg, const GWEN_TAG16_LIST *tagList)
 {
+  if (tagList) {
     AQHOME_SERVER *xo;
   
     xo=AqHomeDataServer_GetServerData(o);
@@ -126,6 +127,7 @@ void AqHomeDataServer_HandleSetData(AQH_OBJECT *o, AQH_OBJECT *epSrc, const AQH_
       }
     }
   }
+}
 
 
 

apps/aqhome-data/s_updatedata.c

@@ -53,6 +53,7 @@ static void _sendDataChangedMsgToAllClients(AQHOME_SERVER *xo, AQH_OBJECT *epSrc
 
 void AqHomeDataServer_HandleUpdateData(AQH_OBJECT *o, AQH_OBJECT *ep, const AQH_MESSAGE *msg, const GWEN_TAG16_LIST *tagList)
 {
+  if (tagList) {
     AQHOME_SERVER *xo;
   
     xo=AqHomeDataServer_GetServerData(o);
@@ -109,6 +110,7 @@ void AqHomeDataServer_HandleUpdateData(AQH_OBJECT *o, AQH_OBJECT *ep, const AQH_
       AQH_Endpoint_AddMsgOut(ep, outMsg);
     }
   }
+}
 
 
 

apps/aqhome-data/server.c

@@ -423,12 +423,11 @@ void _handleMsgsFromClient(AQH_OBJECT *o, AQHOME_SERVER *xo, AQH_OBJECT *ep)
 void _handleMsgFromClient(AQH_OBJECT *o, AQH_OBJECT *ep, const AQH_MESSAGE *msg)
 {
   GWEN_TAG16_LIST *tagList;
-
-  tagList=AQH_IpcMessageTag16_ParsePayload(msg, 0);
-  if (tagList) {
   uint16_t code;
   uint8_t protoId;
 
+  tagList=AQH_IpcMessageTag16_ParsePayload(msg, 0);
+
   code=AQH_IpcMessage_GetCode(msg);
   protoId=AQH_IpcMessage_GetProtoId(msg);
   if (protoId==AQH_IPC_PROTOCOL_DATA_ID) {
@@ -451,7 +450,6 @@ void _handleMsgFromClient(AQH_OBJECT *o, AQH_OBJECT *ep, const AQH_MESSAGE *msg)
   }
   GWEN_Tag16_List_free(tagList);
 }
-}
 
 
 

apps/aqhome-nodes/r_connect.c

@@ -31,6 +31,7 @@
 
 void AQH_NodeServer_HandleConnect(GWEN_UNUSED AQH_OBJECT *o, AQH_OBJECT *ep, const AQH_MESSAGE *msg, const GWEN_TAG16_LIST *tagList)
 {
+  if (tagList) {
     AQH_MESSAGE *outMsg;
     int resultCode=AQH_MSGDATA_RESULT_SUCCESS;
     char *clientId=NULL;
@@ -74,7 +75,7 @@ void AQH_NodeServer_HandleConnect(GWEN_UNUSED AQH_OBJECT *o, AQH_OBJECT *ep, con
                                     AQH_IpcMessage_GetMsgId(msg),
                                     resultCode, NULL);
     AQH_Endpoint_AddMsgOut(ep, outMsg);
-  DBG_ERROR(NULL, "Client connected");
+  }
 }