diff options
| author | Benjamin Dauvergne <bdauvergne@entrouvert.com> | 2010-04-22 00:44:45 +0000 |
|---|---|---|
| committer | Benjamin Dauvergne <bdauvergne@entrouvert.com> | 2010-04-22 00:44:45 +0000 |
| commit | 0a169eda51add0070a68b6280293da03529ae78a (patch) | |
| tree | a85f2e71aeffb17b8dc65cb5aa53a94cf682d796 /lasso/id-wsf/authentication.c | |
| parent | c93fb2a356cd8af36bfaa6f5c356e8ad8824f30d (diff) | |
| download | lasso-0a169eda51add0070a68b6280293da03529ae78a.tar.gz lasso-0a169eda51add0070a68b6280293da03529ae78a.tar.xz lasso-0a169eda51add0070a68b6280293da03529ae78a.zip | |
Improve safety by replacing all g_free use by lasso_release
Diffstat (limited to 'lasso/id-wsf/authentication.c')
| -rw-r--r-- | lasso/id-wsf/authentication.c | 15 |
1 files changed, 8 insertions, 7 deletions
diff --git a/lasso/id-wsf/authentication.c b/lasso/id-wsf/authentication.c index fe54ee16..4707b04d 100644 --- a/lasso/id-wsf/authentication.c +++ b/lasso/id-wsf/authentication.c @@ -22,6 +22,7 @@ * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA */ +#include "../utils.h" #include "./authentication.h" #include "../xml/private.h" #include "../xml/sa_sasl_request.h" @@ -153,7 +154,7 @@ lasso_authentication_client_start(LassoAuthentication *authentication) /* mechusing is th resulting best mech to use, so copy it in SASLRequest element */ if (mechusing != NULL) { - g_free(request->mechanism); + lasso_release(request->mechanism); request->mechanism = g_strdup(mechusing); } @@ -531,13 +532,13 @@ lasso_authentication_server_start(LassoAuthentication *authentication) } } if (chosen == NULL) { - g_free(response->Status->code); + lasso_release(response->Status->code); response->Status->code = g_strdup(LASSO_SA_STATUS_CODE_ABORT); return res; } if (nbmech > 1 && request->Data != NULL) { - g_free(response->Status->code); + lasso_release(response->Status->code); response->Status->code = g_strdup(LASSO_SA_STATUS_CODE_ABORT); return res; } @@ -560,7 +561,7 @@ lasso_authentication_server_start(LassoAuthentication *authentication) /* set status code in SASLResponse message if not ok */ if (res != SASL_OK) { - g_free(response->Status->code); + lasso_release(response->Status->code); /* continue, set Data in response */ if (res == SASL_CONTINUE) { @@ -602,7 +603,7 @@ lasso_authentication_server_step(LassoAuthentication *authentication) /* If mechanism is NULL, thene client wants to abort authentication exchange */ if (g_str_equal(request->mechanism, "") == TRUE) { - g_free(response->Status->code); + lasso_release(response->Status->code); response->Status->code = g_strdup(LASSO_SA_STATUS_CODE_ABORT); return 0; @@ -621,7 +622,7 @@ lasso_authentication_server_step(LassoAuthentication *authentication) &outlen); if (res != SASL_OK) { - g_free(response->Status->code); + lasso_release(response->Status->code); if (res == SASL_CONTINUE) { /* authentication exchange must continue */ @@ -694,7 +695,7 @@ static void finalize(GObject *object) { LassoAuthentication *authentication = LASSO_AUTHENTICATION(object); - g_free(authentication->private_data); + lasso_release(authentication->private_data); authentication->private_data = NULL; G_OBJECT_CLASS(parent_class)->finalize(object); } |