Revert "fix(合并mqtt序列化库): Fix potential overwrites eclipse-paho/paho.m…

…qtt.embedded-c#242"

This reverts commit 5a7b046.

pahoMqtt/MQTTUnsubscribeServer.c

@@ -1,5 +1,5 @@
 /*******************************************************************************
- * Copyright (c) 2014, 2023 IBM Corp., Ian Craggs
+ * Copyright (c) 2014 IBM Corp.
  *
  * All rights reserved. This program and the accompanying materials
  * are made available under the terms of the Eclipse Public License v1.0
@@ -21,51 +21,41 @@
 
 
 /**
-  * Deserializes the supplied (wire) buffer into subscribe data
+  * Deserializes the supplied (wire) buffer into unsubscribe data
   * @param dup integer returned - the MQTT dup flag
   * @param packetid integer returned - the MQTT packet identifier
   * @param maxcount - the maximum number of members allowed in the topicFilters and requestedQoSs arrays
   * @param count - number of members in the topicFilters and requestedQoSs arrays
   * @param topicFilters - array of topic filter names
-  * @param requestedQoSs - array of requested QoS
   * @param buf the raw buffer data, of the correct length determined by the remaining length field
   * @param buflen the length in bytes of the data in the supplied buffer
   * @return the length of the serialized data.  <= 0 indicates error
   */
-int MQTTDeserialize_subscribe(unsigned char* dup, unsigned short* packetid, int maxcount, int* count, MQTTString topicFilters[],
-	int requestedQoSs[], unsigned char* buf, int buflen)
+int MQTTDeserialize_unsubscribe(unsigned char* dup, unsigned short* packetid, int maxcount, int* count, MQTTString topicFilters[],
+		unsigned char* buf, int len)
 {
 	MQTTHeader header = {0};
 	unsigned char* curdata = buf;
 	unsigned char* enddata = NULL;
-	int rc = MQTTPACKET_READ_ERROR;
+	int rc = 0;
 	int mylen = 0;
 
 	FUNC_ENTRY;
 	header.byte = readChar(&curdata);
-	if (header.bits.type != SUBSCRIBE)
+	if (header.bits.type != UNSUBSCRIBE)
 		goto exit;
 	*dup = header.bits.dup;
 
-	rc = MQTTPacket_decodeBuf(curdata, &mylen); /* read remaining length */
-	if (rc <= 0)
-		goto exit;
-	curdata += rc;
-	rc = MQTTPACKET_READ_ERROR;
+	curdata += (rc = MQTTPacket_decodeBuf(curdata, &mylen)); /* read remaining length */
 	enddata = curdata + mylen;
 
 	*packetid = readInt(&curdata);
 
 	*count = 0;
 	while (curdata < enddata)
 	{
-		if (*count == maxcount)
-			goto exit;
 		if (!readMQTTLenString(&topicFilters[*count], &curdata, enddata))
 			goto exit;
-		if (curdata >= enddata) /* do we have enough data to read the req_qos version byte? */
-			goto exit;
-		requestedQoSs[*count] = readChar(&curdata);
 		(*count)++;
 	}
 
@@ -77,41 +67,36 @@ int MQTTDeserialize_subscribe(unsigned char* dup, unsigned short* packetid, int
 
 
 /**
-  * Serializes the supplied suback data into the supplied buffer, ready for sending
+  * Serializes the supplied unsuback data into the supplied buffer, ready for sending
   * @param buf the buffer into which the packet will be serialized
   * @param buflen the length in bytes of the supplied buffer
   * @param packetid integer - the MQTT packet identifier
-  * @param count - number of members in the grantedQoSs array
-  * @param grantedQoSs - array of granted QoS
   * @return the length of the serialized data.  <= 0 indicates error
   */
-int MQTTSerialize_suback(unsigned char* buf, int buflen, unsigned short packetid, int count, int* grantedQoSs)
+int MQTTSerialize_unsuback(unsigned char* buf, int buflen, unsigned short packetid)
 {
 	MQTTHeader header = {0};
-	int rc = -1;
+	int rc = 0;
 	unsigned char *ptr = buf;
-	int i;
 
 	FUNC_ENTRY;
-	if (buflen < 2 + count)
+	if (buflen < 2)
 	{
 		rc = MQTTPACKET_BUFFER_TOO_SHORT;
 		goto exit;
 	}
 	header.byte = 0;
-	header.bits.type = SUBACK;
+	header.bits.type = UNSUBACK;
 	writeChar(&ptr, header.byte); /* write header */
 
-	ptr += MQTTPacket_encode(ptr, 2 + count); /* write remaining length */
+	ptr += MQTTPacket_encode(ptr, 2); /* write remaining length */
 
 	writeInt(&ptr, packetid);
 
-	for (i = 0; i < count; ++i)
-		writeChar(&ptr, grantedQoSs[i]);
-
 	rc = ptr - buf;
 exit:
 	FUNC_EXIT_RC(rc);
 	return rc;
 }
 
+