--- libaitmqtt/src/sub.c 2012/01/26 13:07:33 1.1 +++ libaitmqtt/src/sub.c 2012/04/27 08:12:30 1.1.1.1.2.4 @@ -3,7 +3,7 @@ * by Michael Pounov * * $Author: misho $ -* $Id: sub.c,v 1.1 2012/01/26 13:07:33 misho Exp $ +* $Id: sub.c,v 1.1.1.1.2.4 2012/04/27 08:12:30 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; } @@ -111,7 +109,6 @@ mqtt_msgSUBSCRIBE(mqtt_msg_t * __restrict buf, mqtt_su hdr->mqtt_msg.retain = 0; *hdr->mqtt_len = mqtt_encodeLen(siz - sizeof(struct mqtthdr)); - mqtt_msgRealloc(buf, siz); return siz; } @@ -159,7 +156,6 @@ mqtt_msgSUBACK(mqtt_msg_t * __restrict buf, mqtt_subsc hdr->mqtt_msg.type = MQTT_TYPE_SUBACK; *hdr->mqtt_len = mqtt_encodeLen(siz - sizeof(struct mqtthdr)); - mqtt_msgRealloc(buf, siz); return siz; } @@ -186,11 +182,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; } @@ -222,7 +218,6 @@ mqtt_msgUNSUBSCRIBE(mqtt_msg_t * __restrict buf, mqtt_ hdr->mqtt_msg.retain = 0; *hdr->mqtt_len = mqtt_encodeLen(siz - sizeof(struct mqtthdr)); - mqtt_msgRealloc(buf, siz); return siz; } @@ -272,9 +267,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 +281,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 +301,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 +311,12 @@ 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) { + if (!mqtt_subRealloc(&subs, i + 1)) { mqtt_subFree(subscr); - return NULL; + return -1; } else *subscr = subs; @@ -332,7 +326,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 +335,7 @@ mqtt_readSUBSCRIBE(mqtt_msg_t * __restrict buf, u_shor pos++; } - return hdr; + return i; } /* @@ -372,7 +366,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 +390,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 +404,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 +424,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 +434,12 @@ 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) { + if (!mqtt_subRealloc(&subs, i + 1)) { mqtt_subFree(subscr); - return NULL; + return -1; } else *subscr = subs; @@ -456,13 +449,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 +476,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;