diff options
| author | dequis <dx@dxzone.com.ar> | 2016-12-26 19:39:18 -0300 |
|---|---|---|
| committer | dequis <dx@dxzone.com.ar> | 2016-12-26 19:39:18 -0300 |
| commit | d57484d99125eb602e71629d7cacf76fc36cdd9c (patch) | |
| tree | 4ca56875a5b22ee30505c8ffad847c50cabf3950 /protocols/oscar/rxhandlers.c | |
| parent | 90a45b8fc30a0b8b96c60926e638291fc0d0fa08 (diff) | |
Change some asserts into g_return_if_fail()
Because crashing asserts are bad, and maybe this helps fix the
captures_build_path issue with debian's reproducible builds
(those asserts probably include __FILE__)
Diffstat (limited to 'protocols/oscar/rxhandlers.c')
| -rw-r--r-- | protocols/oscar/rxhandlers.c | 10 |
1 files changed, 2 insertions, 8 deletions
diff --git a/protocols/oscar/rxhandlers.c b/protocols/oscar/rxhandlers.c index 6ff106b2..0f1bb3d7 100644 --- a/protocols/oscar/rxhandlers.c +++ b/protocols/oscar/rxhandlers.c @@ -234,14 +234,8 @@ int aim_conn_addhandler(aim_session_t *sess, aim_conn_t *conn, guint16 family, g { struct aim_rxcblist_s *newcb; - if (!conn) { - return -1; - } - - if (checkdisallowed(family, type)) { - g_assert(0); - return -1; - } + g_return_val_if_fail(conn, -1); + g_return_val_if_fail(checkdisallowed(family, type), -1); if (!(newcb = (struct aim_rxcblist_s *) g_new0(struct aim_rxcblist_s, 1))) { return -1; |