diff options
| author | dimitri staessens <[email protected]> | 2017-02-21 15:58:29 +0100 |
|---|---|---|
| committer | dimitri staessens <[email protected]> | 2017-02-21 15:58:29 +0100 |
| commit | 2a73f2d58cd29c6ecaade828db6ce1f5dde18a0e (patch) | |
| tree | ccfccdbf0cb9dfd3db0f754660205eeb3a9eac7a /src/lib/pol/cacep_simple_auth.c | |
| parent | 198e319d14af53891b81cca36fe92e06d5d7d67d (diff) | |
| download | ouroboros-2a73f2d58cd29c6ecaade828db6ce1f5dde18a0e.tar.gz ouroboros-2a73f2d58cd29c6ecaade828db6ce1f5dde18a0e.zip | |
lib: Clean up memory management in cacep policies
Diffstat (limited to 'src/lib/pol/cacep_simple_auth.c')
| -rw-r--r-- | src/lib/pol/cacep_simple_auth.c | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/src/lib/pol/cacep_simple_auth.c b/src/lib/pol/cacep_simple_auth.c index b24a818b..65c510a2 100644 --- a/src/lib/pol/cacep_simple_auth.c +++ b/src/lib/pol/cacep_simple_auth.c @@ -59,6 +59,8 @@ static struct cacep_info * read_msg(int fd) return NULL; } + cacep_info_init(tmp); + tmp->addr = msg->addr; tmp->name = strdup(msg->name); if (tmp->name == NULL) { @@ -78,8 +80,7 @@ static struct cacep_info * read_msg(int fd) tmp->proto.pref_version = msg->proto->pref_version; tmp->proto.pref_syntax = code_to_syntax(msg->proto->pref_syntax); if (tmp->proto.pref_syntax < 0) { - free(tmp->proto.protocol); - free(tmp->name); + cacep_info_fini(tmp); free(tmp); cacep_simple_auth_msg__free_unpacked(msg, NULL); return NULL; @@ -144,6 +145,7 @@ struct cacep_info * cacep_simple_auth_auth(int fd, if (strcmp(info->proto.protocol, tmp->proto.protocol) || info->proto.pref_version != tmp->proto.pref_version || info->proto.pref_syntax != tmp->proto.pref_syntax) { + cacep_info_fini(tmp); free(tmp); return NULL; } @@ -164,6 +166,7 @@ struct cacep_info * cacep_simple_auth_auth_wait(int fd, return NULL; if (send_msg(fd, info)) { + cacep_info_fini(tmp); free(tmp); return NULL; } @@ -171,6 +174,7 @@ struct cacep_info * cacep_simple_auth_auth_wait(int fd, if (strcmp(info->proto.protocol, tmp->proto.protocol) || info->proto.pref_version != tmp->proto.pref_version || info->proto.pref_syntax != tmp->proto.pref_syntax) { + cacep_info_fini(tmp); free(tmp); return NULL; } |