Changeset d57484d for protocols/oscar/rxhandlers.c

Timestamp:

2016-12-26T22:39:18Z (8 years ago)

Author:

dequis <dx@…>

Branches:

master

Children:

a04705b

Parents:

90a45b8

Message:

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)

File:

• protocols/oscar/rxhandlers.c (modified) (1 diff)

protocols/oscar/rxhandlers.c

@@ -235 +235 @@
         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))) {