--- libaitmqtt/src/sub.c 2012/01/26 13:07:33 1.1.1.1 +++ libaitmqtt/src/sub.c 2012/04/26 11:49:12 1.1.1.1.2.2 @@ -3,7 +3,7 @@ * by Michael Pounov * * $Author: misho $ -* $Id: sub.c,v 1.1.1.1 2012/01/26 13:07:33 misho Exp $ +* $Id: sub.c,v 1.1.1.1.2.2 2012/04/26 11:49:12 misho Exp $ * ************************************************************************** The ELWIX and AITNET software is distributed under the following @@ -12,7 +12,7 @@ terms: All of the documentation and software included in the ELWIX and AITNET Releases is copyrighted by ELWIX - Sofia/Bulgaria -Copyright 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2011 +Copyright 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2011, 2012 by Michael Pounov . All rights reserved. Redistribution and use in source and binary forms, with or without @@ -46,8 +46,6 @@ SUCH DAMAGE. #include "global.h" -/* ------------------------------------------------------------------- */ - /* * mqtt_msgSUBSCRIBE() Create SUBSCRIBE message * @@ -72,11 +70,11 @@ mqtt_msgSUBSCRIBE(mqtt_msg_t * __restrict buf, mqtt_su if (!buf || !Topics) return -1; if (QOS > MQTT_QOS_EXACTLY) { - mqtt_SetErr(EINVAL, "Error:: invalid QoS parameter"); + mqtt_SetErr(EINVAL, "Invalid QoS parameter"); return -1; } if (!msgID && QOS != MQTT_QOS_ONCE) { - mqtt_SetErr(EINVAL, "Error:: invalid MessageID parameter must be >0"); + mqtt_SetErr(EINVAL, "Invalid MessageID parameter must be >0"); return -1; } @@ -186,11 +184,11 @@ mqtt_msgUNSUBSCRIBE(mqtt_msg_t * __restrict buf, mqtt_ if (!buf || !Topics) return -1; if (QOS > MQTT_QOS_EXACTLY) { - mqtt_SetErr(EINVAL, "Error:: invalid QoS parameter"); + mqtt_SetErr(EINVAL, "Invalid QoS parameter"); return -1; } if (!msgID && QOS != MQTT_QOS_ONCE) { - mqtt_SetErr(EINVAL, "Error:: invalid MessageID parameter must be >0"); + mqtt_SetErr(EINVAL, "Invalid MessageID parameter must be >0"); return -1; } @@ -272,9 +270,9 @@ mqtt_msgUNSUBACK(mqtt_msg_t * __restrict buf, u_short * @buf = Message buffer * @msgID = MessageID * @subscr = Subscriptions, must be free after use with mqtt_subFree() - * return: NULL error or !=NULL MQTT fixed header + * return: -1 error or >-1 elements into subscr */ -struct mqtthdr * +int mqtt_readSUBSCRIBE(mqtt_msg_t * __restrict buf, u_short *msgID, mqtt_subscr_t **subscr) { register int i; @@ -286,19 +284,19 @@ mqtt_readSUBSCRIBE(mqtt_msg_t * __restrict buf, u_shor caddr_t pos; if (!buf || !msgID || !subscr) - return NULL; + return -1; hdr = _mqtt_readHEADER(buf, MQTT_TYPE_SUBSCRIBE, &ret, &len); if (!hdr) - return NULL; + return -1; pos = buf->msg_base + ret + 1; v = (mqtt_len_t*) pos; /* MessageID */ len -= sizeof(mqtt_len_t); if (len < 0) { - mqtt_SetErr(EINVAL, "Error:: short message length %d", len); - return NULL; + mqtt_SetErr(EINVAL, "Short message length %d", len); + return -1; } else { *msgID = ntohs(v->val); pos += sizeof(mqtt_len_t); @@ -306,7 +304,7 @@ mqtt_readSUBSCRIBE(mqtt_msg_t * __restrict buf, u_shor subs = mqtt_subAlloc(0); if (!subs) - return NULL; + return -1; else *subscr = subs; @@ -316,13 +314,13 @@ mqtt_readSUBSCRIBE(mqtt_msg_t * __restrict buf, u_shor len -= MQTTHDR_VAR_SIZEOF(var) + 1; if (len < 0) { mqtt_subFree(subscr); - mqtt_SetErr(EINVAL, "Error:: short message length %d", len); - return NULL; + mqtt_SetErr(EINVAL, "Short message length %d", len); + return -1; } subs = mqtt_subRealloc(subs, i + 1); if (!subs) { mqtt_subFree(subscr); - return NULL; + return -1; } else *subscr = subs; @@ -332,7 +330,7 @@ mqtt_readSUBSCRIBE(mqtt_msg_t * __restrict buf, u_shor if (!subs[i].sub_topic.msg_base) { LOGERR; mqtt_subFree(subscr); - return NULL; + return -1; } else memcpy(subs[i].sub_topic.msg_base, var->var_data, subs[i].sub_topic.msg_len); pos += MQTTHDR_VAR_SIZEOF(var); @@ -341,7 +339,7 @@ mqtt_readSUBSCRIBE(mqtt_msg_t * __restrict buf, u_shor pos++; } - return hdr; + return i; } /* @@ -372,7 +370,7 @@ mqtt_readSUBACK(mqtt_msg_t * __restrict buf, u_short * /* MessageID */ len -= sizeof(mqtt_len_t); if (len < 0) { - mqtt_SetErr(EINVAL, "Error:: short message length %d", len); + mqtt_SetErr(EINVAL, "Short message length %d", len); return -1; } else { *msgID = ntohs(v->val); @@ -396,9 +394,9 @@ mqtt_readSUBACK(mqtt_msg_t * __restrict buf, u_short * * @buf = Message buffer * @msgID = MessageID * @subscr = Subscriptions, must be free after use with mqtt_subFree() - * return: NULL error or !=NULL MQTT fixed header + * return: -1 error or >-1 elements into subscr */ -struct mqtthdr * +int mqtt_readUNSUBSCRIBE(mqtt_msg_t * __restrict buf, u_short *msgID, mqtt_subscr_t **subscr) { register int i; @@ -410,19 +408,19 @@ mqtt_readUNSUBSCRIBE(mqtt_msg_t * __restrict buf, u_sh caddr_t pos; if (!buf || !msgID || !subscr) - return NULL; + return -1; hdr = _mqtt_readHEADER(buf, MQTT_TYPE_UNSUBSCRIBE, &ret, &len); if (!hdr) - return NULL; + return -1; pos = buf->msg_base + ret + 1; v = (mqtt_len_t*) pos; /* MessageID */ len -= sizeof(mqtt_len_t); if (len < 0) { - mqtt_SetErr(EINVAL, "Error:: short message length %d", len); - return NULL; + mqtt_SetErr(EINVAL, "Short message length %d", len); + return -1; } else { *msgID = ntohs(v->val); pos += sizeof(mqtt_len_t); @@ -430,7 +428,7 @@ mqtt_readUNSUBSCRIBE(mqtt_msg_t * __restrict buf, u_sh subs = mqtt_subAlloc(0); if (!subs) - return NULL; + return -1; else *subscr = subs; @@ -440,13 +438,13 @@ mqtt_readUNSUBSCRIBE(mqtt_msg_t * __restrict buf, u_sh len -= MQTTHDR_VAR_SIZEOF(var); if (len < 0) { mqtt_subFree(subscr); - mqtt_SetErr(EINVAL, "Error:: short message length %d", len); - return NULL; + mqtt_SetErr(EINVAL, "Short message length %d", len); + return -1; } subs = mqtt_subRealloc(subs, i + 1); if (!subs) { mqtt_subFree(subscr); - return NULL; + return -1; } else *subscr = subs; @@ -456,13 +454,13 @@ mqtt_readUNSUBSCRIBE(mqtt_msg_t * __restrict buf, u_sh if (!subs[i].sub_topic.msg_base) { LOGERR; mqtt_subFree(subscr); - return NULL; + return -1; } else memcpy(subs[i].sub_topic.msg_base, var->var_data, subs[i].sub_topic.msg_len); pos += MQTTHDR_VAR_SIZEOF(var); } - return hdr; + return i; } /* @@ -483,7 +481,7 @@ mqtt_readUNSUBACK(mqtt_msg_t * __restrict buf) if (!hdr) return (u_short) -1; if (len < sizeof(mqtt_len_t)) { - mqtt_SetErr(EINVAL, "Error:: short message length %d", len); + mqtt_SetErr(EINVAL, "Short message length %d", len); return (u_short) -1; } else { pos = buf->msg_base + ret + 1;