diff options
| author | OBATA Akio <obache@wizdas.com> | 2020-02-03 11:40:27 +0900 |
|---|---|---|
| committer | OBATA Akio <obache@wizdas.com> | 2020-02-03 14:02:03 +0900 |
| commit | 8d21538ce07e8c91023fd8d4997563d7db156ed6 (patch) | |
| tree | 80a8116e7fdaaf75d51ce87659f1d1238891f00f | |
| parent | 362850bf60baa8fc6ea1f88f073423c2666bc828 (diff) | |
| download | libcarddav-8d21538ce07e8c91023fd8d4997563d7db156ed6.tar.gz libcarddav-8d21538ce07e8c91023fd8d4997563d7db156ed6.zip | |
Remove unwanted memory allocation for response
It is expected as pointer to struct response with size of the struct,
not pointer to the pointer, but never returned to caller.
It should be allocated by caller.
Signed-off-by: OBATA Akio <obache@wizdas.com>
| -rw-r--r-- | src/carddav.c | 20 |
1 files changed, 4 insertions, 16 deletions
diff --git a/src/carddav.c b/src/carddav.c index aeb0ad5..92e6362 100644 --- a/src/carddav.c +++ b/src/carddav.c @@ -425,12 +425,9 @@ CARDDAV_RESPONSE carddav_get_object(response *result, CARDDAV_RESPONSE carddav_response; g_return_val_if_fail(info != NULL, TRUE); + g_return_val_if_fail(result != NULL, NOTIMPLEMENTED); init_runtime(info); - if (!result) { - result = malloc(sizeof(response *)); - memset(result, '\0', sizeof(response *)); - } init_carddav_settings(&settings); settings.ACTION = GET; settings.start = start; @@ -489,12 +486,9 @@ CARDDAV_RESPONSE carddav_getall_object(response* result, CARDDAV_RESPONSE carddav_response; g_return_val_if_fail(info != NULL, TRUE); + g_return_val_if_fail(result != NULL, NOTIMPLEMENTED); init_runtime(info); - if (!result) { - result = malloc(sizeof(response *)); - memset(result, '\0', sizeof(response *)); - } init_carddav_settings(&settings); settings.ACTION = GETALL; if (info->options->debug) @@ -553,12 +547,9 @@ CARDDAV_RESPONSE carddav_getall_object_by_uri(response* result, CARDDAV_RESPONSE carddav_response; g_return_val_if_fail(info != NULL, TRUE); + g_return_val_if_fail(result != NULL, NOTIMPLEMENTED); init_runtime(info); - if (!result) { - result = malloc(sizeof(response *)); - memset(result, '\0', sizeof(response *)); - } init_carddav_settings(&settings); settings.ACTION = GETALL; if (info->options->debug) @@ -616,12 +607,9 @@ CARDDAV_RESPONSE carddav_get_displayname(response* result, CARDDAV_RESPONSE carddav_response; g_return_val_if_fail(info != NULL, TRUE); + g_return_val_if_fail(result != NULL, NOTIMPLEMENTED); init_runtime(info); - if (!result) { - result = malloc(sizeof(response *)); - memset(result, '\0', sizeof(response *)); - } init_carddav_settings(&settings); settings.ACTION = GETCALNAME; if (info->options->debug) |