*_free methods are used in error cases
This commit is contained in:
parent
18a589ec46
commit
d60f570f0d
13 changed files with 13 additions and 12 deletions
|
@ -49,7 +49,7 @@ sml_attention_response *sml_attention_response_parse(sml_buffer *buf){
|
|||
return msg;
|
||||
|
||||
error:
|
||||
free(msg);
|
||||
sml_attention_response_free(msg);
|
||||
return 0;
|
||||
|
||||
}
|
||||
|
|
|
@ -55,7 +55,7 @@ sml_close_request * sml_close_request_parse(sml_buffer *buf) {
|
|||
return msg;
|
||||
|
||||
error:
|
||||
free(msg);
|
||||
sml_close_request_free(msg);
|
||||
return 0;
|
||||
}
|
||||
|
||||
|
|
|
@ -41,7 +41,7 @@ sml_close_response *sml_close_response_parse(sml_buffer *buf) {
|
|||
return msg;
|
||||
|
||||
error:
|
||||
free(msg);
|
||||
sml_close_response_free(msg);
|
||||
return 0;
|
||||
|
||||
}
|
||||
|
|
|
@ -88,7 +88,7 @@ sml_get_list_request *sml_get_list_request_parse(sml_buffer *buf) {
|
|||
return msg;
|
||||
|
||||
error:
|
||||
free(msg);
|
||||
sml_get_list_request_free(msg);
|
||||
return 0;
|
||||
|
||||
}
|
||||
|
|
|
@ -60,7 +60,7 @@ sml_get_list_response *sml_get_list_response_parse(sml_buffer *buf) {
|
|||
return msg;
|
||||
|
||||
error:
|
||||
free(msg);
|
||||
sml_get_list_response_free(msg);
|
||||
return 0;
|
||||
|
||||
}
|
||||
|
|
|
@ -112,7 +112,7 @@ sml_get_profile_pack_request *sml_get_profile_pack_request_parse(sml_buffer *buf
|
|||
return msg;
|
||||
|
||||
error:
|
||||
free(msg);
|
||||
sml_get_profile_pack_request_free(msg);
|
||||
return 0;
|
||||
|
||||
}
|
||||
|
|
|
@ -96,6 +96,7 @@ sml_list *sml_list_parse(sml_buffer *buf) {
|
|||
|
||||
error:
|
||||
buf->error = 1;
|
||||
sml_list_free(start);
|
||||
return 0;
|
||||
}
|
||||
|
||||
|
|
|
@ -74,7 +74,7 @@ sml_message *sml_message_parse(sml_buffer *buf) {
|
|||
|
||||
error:
|
||||
// TODO: free encapsulated stuff
|
||||
free(msg);
|
||||
sml_message_free(msg);
|
||||
return 0;
|
||||
}
|
||||
|
||||
|
|
|
@ -106,7 +106,7 @@ sml_open_request *sml_open_request_parse(sml_buffer *buf) {
|
|||
return msg;
|
||||
|
||||
error:
|
||||
free(msg);
|
||||
sml_open_request_free(msg);
|
||||
return 0;
|
||||
}
|
||||
|
||||
|
|
|
@ -55,7 +55,7 @@ sml_open_response *sml_open_response_parse(sml_buffer *buf) {
|
|||
|
||||
return msg;
|
||||
error:
|
||||
free(msg);
|
||||
sml_open_response_free(msg);
|
||||
return 0;
|
||||
}
|
||||
|
||||
|
|
|
@ -54,7 +54,7 @@ sml_status *sml_status_parse(sml_buffer *buf) {
|
|||
break;
|
||||
}
|
||||
if (sml_buf_has_errors(buf)) {
|
||||
free(state);
|
||||
sml_status_free(state);
|
||||
return 0;
|
||||
}
|
||||
|
||||
|
|
|
@ -44,7 +44,7 @@ sml_time *sml_time_parse(sml_buffer *buf) {
|
|||
return tme;
|
||||
|
||||
error:
|
||||
free(tme);
|
||||
sml_time_free(tme);
|
||||
return 0;
|
||||
}
|
||||
|
||||
|
|
|
@ -96,7 +96,7 @@ sml_value *sml_value_parse(sml_buffer *buf) {
|
|||
break;
|
||||
}
|
||||
if (sml_buf_has_errors(buf)) {
|
||||
free(value);
|
||||
sml_value_free(value);
|
||||
return 0;
|
||||
}
|
||||
|
||||
|
|
Loading…
Add table
Reference in a new issue