diff options
| author | Benjamin Dauvergne <bdauvergne@entrouvert.com> | 2010-01-14 16:18:29 +0000 |
|---|---|---|
| committer | Benjamin Dauvergne <bdauvergne@entrouvert.com> | 2010-01-14 16:18:29 +0000 |
| commit | f87a55e9ef09d79c0b1ec027a0fa29d2fe1f3c85 (patch) | |
| tree | 0e7ce396773fd1cd72f76147d2c9c7854bf01726 | |
| parent | 9d544c49d0610e421bf269cccc71e720a53eccde (diff) | |
| download | lasso-f87a55e9ef09d79c0b1ec027a0fa29d2fe1f3c85.tar.gz lasso-f87a55e9ef09d79c0b1ec027a0fa29d2fe1f3c85.tar.xz lasso-f87a55e9ef09d79c0b1ec027a0fa29d2fe1f3c85.zip | |
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.
| -rw-r--r-- | lasso/id-ff/session.c | 5 |
1 files 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; |