# receiving corrupt data with xQueueReceive

**URL:** <https://forums.freertos.org/t/receiving-corrupt-data-with-xqueuereceive/764>\
**Category:** Kernel\
**Created:** [September 30, 2012, 11:26pm UTC](https://forums.freertos.org/t/receiving-corrupt-data-with-xqueuereceive/764 "2012-09-30T23:26:08Z")\
**Posts on this page:** 3\
**Page:** 1

<div class="post-metadata">

**Author:** ![system](https://d2v6vdsk2p900z.cloudfront.net/original/1X/7fc8fdb729c645aab5b20282bbe34e57cb407be6.jpeg) [@system](https://forums.freertos.org/u/system)\
**Post date:** [September 30, 2012, 11:26pm UTC](https://forums.freertos.org/t/receiving-corrupt-data-with-xqueuereceive/764/1 "2012-09-30T23:26:08Z")

</div>

**[modiallen](https://sourceforge.net/u/modiallen/)** wrote on [Sunday, September 30, 2012](https://sourceforge.net/p/freertos/discussion/382005/thread/20420fa0/#e86e):

Hi everyone,  
i have the following problem:  
i dont receive the same data with xQueueReceive, that i sent it with the xQueueSend.  
here is the code, maybe someone can help me solving the problem:

void privateMenuCreateRefreshEvent(unsigned char pattern, unsigned char field, unsigned int sensor, unsigned char value)  
{  
struct Display\_Message SendMessage;  
struct DataFieldUpdate \*pointer = malloc(sizeof(struct DataFieldUpdate));  
pointer-\>fieldNumber = 0;  
pointer-\>patternType = 0;  
pointer-\>valueID = 0;  
pointer-\>valuetype = 0;  
SendMessage.DataFieldObject = pointer;  
SendMessage.Msg\_Type = ProtocolData\_Object;

xQueueSend(xDisplayQueue,&SendMessage,0);  
free(pointer);  
}

and the receiving task:

void dplTask(void \*pvParameters)  
{

for (;😉  
{  
struct Display\_Message \*ReceivedMessage = malloc(sizeof(struct Display\_Message));;  
xQueueReceive(xDisplayQueue,ReceivedMessage,portMAX\_DELAY);  
if ( ReceivedMessage-\>Msg\_Type == ProtocolData\_Object )  
{  
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; dplUpdateDisplayExecuteFunction(ReceivedMessage-\>DataFieldObject);  
free(ReceivedMessage);  
}  
}  
}

the ReceivedMessage-\>DataFieldObject dont have the same values that are sent in the example above.

what could be the problem here.

Best Regards.

---

<div class="post-metadata">

**Author:** ![richard-damon](https://d3iykfbgfh8zhg.cloudfront.net/user_avatar/forums.freertos.org/richard-damon/32/38_2.png) [@richard-damon](https://forums.freertos.org/u/richard-damon)\
**Post date:** [October 1, 2012, 1:30am UTC](https://forums.freertos.org/t/receiving-corrupt-data-with-xqueuereceive/764/2 "2012-10-01T01:30:20Z")

</div>

**[richard\_damon](https://sourceforge.net/u/richard_damon/)** wrote on [Monday, October 01, 2012](https://sourceforge.net/p/freertos/discussion/382005/thread/20420fa0/#ca54):

When you queue up SendMessage, the contents of the structure itself are copied into the queue, which will include the value of the pointer SendMessage.DataFieldObject. It does NOT make a copy of the DataFieldUpdate object. When you then do the free(pointer) the memory holding the values are returned to the heap, and possibly overwritten.

You should NOT do the free(pointer) at the sending side, but leave the DataFieldUpdate object on the heap, and when the data is received and processed, you then free the object on the receive side before the free(ReceviedMessage).

---

<div class="post-metadata">

**Author:** ![system](https://d2v6vdsk2p900z.cloudfront.net/original/1X/7fc8fdb729c645aab5b20282bbe34e57cb407be6.jpeg) [@system](https://forums.freertos.org/u/system)\
**Post date:** [October 1, 2012, 2:04pm UTC](https://forums.freertos.org/t/receiving-corrupt-data-with-xqueuereceive/764/3 "2012-10-01T14:04:49Z")

</div>

**[modiallen](https://sourceforge.net/u/modiallen/)** wrote on [Monday, October 01, 2012](https://sourceforge.net/p/freertos/discussion/382005/thread/20420fa0/#f8c7):

thank you very much richard.  
i changed the struct Display\_Message SendMessage and made the DataFieldObject in it not as a pointer. because i need about 30 queue-item for this queue and some items must wait until they will be processed. and now it is working very well.  
thanks again.