jyapayne/moonlight-common-c
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

Add FLAG_ALLOCATED_PAYLOAD to tell freeMessage() to also free the payload buffer. Don't free the message itself in freeMessage() because it's always stack allocated

Browse source
This commit is contained in:
Cameron Gutman 2014-08-17 19:08:32 -07:00
commit fb40201672
2 changed files with 6 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

1
limelight-common/Rtsp.h
View file

@ -16,6 +16,7 @@
#define FLAG_ALLOCATED_OPTION_FIELDS 0x1 #define FLAG_ALLOCATED_OPTION_FIELDS 0x1
#define FLAG_ALLOCATED_MESSAGE_BUFFER 0x2 #define FLAG_ALLOCATED_MESSAGE_BUFFER 0x2
#define FLAG_ALLOCATED_OPTION_ITEMS 0x4 #define FLAG_ALLOCATED_OPTION_ITEMS 0x4
#define FLAG_ALLOCATED_PAYLOAD 0x8
/* Linked List to store the options */ /* Linked List to store the options */
typedef struct _OPTION_ITEM { typedef struct _OPTION_ITEM {

6
limelight-common/RtspParser.c
View file

@ -353,5 +353,9 @@ void freeMessage(PRTSP_MESSAGE msg){
if (msg->flags & FLAG_ALLOCATED_OPTION_ITEMS){ if (msg->flags & FLAG_ALLOCATED_OPTION_ITEMS){
freeOptionList(msg->options); freeOptionList(msg->options);
} }
free(msg);
/* If we've allocated the payload */
if (msg->flags & FLAG_ALLOCATED_PAYLOAD) {
free(msg->payload);
}
} }