From f87a55e9ef09d79c0b1ec027a0fa29d2fe1f3c85 Mon Sep 17 00:00:00 2001 From: Benjamin Dauvergne Date: Thu, 14 Jan 2010 16:18:29 +0000 Subject: Core: remove warning emiting macros from lasso_session_remove_assertion * lasso/id-ff/session.c: we already return error code, no need to log more warnings. --- lasso/id-ff/session.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/lasso/id-ff/session.c b/lasso/id-ff/session.c index 0726306b..113ae47a 100644 --- a/lasso/id-ff/session.c +++ b/lasso/id-ff/session.c @@ -382,8 +382,9 @@ lasso_session_is_dirty(LassoSession *session) gint lasso_session_remove_assertion(LassoSession *session, const gchar *providerID) { - g_return_val_if_fail(LASSO_IS_SESSION(session), LASSO_PARAM_ERROR_INVALID_VALUE); - g_return_val_if_fail(providerID != NULL, LASSO_PARAM_ERROR_INVALID_VALUE); + if (! LASSO_IS_SESSION(session) || lasso_is_empty_string(providerID)) { + return LASSO_PARAM_ERROR_INVALID_VALUE; + } if (g_hash_table_remove(session->assertions, providerID)) { session->is_dirty = TRUE; -- cgit