Refactor: based: Drop redundant NULL checks

pcmk__drop_all_clients() returns immediately if its argument is NULL,
and g_clear_pointer() is NULL-safe.

Signed-off-by: Reid Wahl <nrwahl@protonmail.com>

Author: nrwahl2

daemons/based/based_callbacks.c

@@ -913,20 +913,14 @@ based_shutdown(int nsig)
 
     cib_shutdown_flag = true;
 
-    if (ipcs_ro != NULL) {
-        pcmk__drop_all_clients(ipcs_ro);
-        g_clear_pointer(&ipcs_ro, qb_ipcs_destroy);
-    }
+    pcmk__drop_all_clients(ipcs_ro);
+    g_clear_pointer(&ipcs_ro, qb_ipcs_destroy);
 
-    if (ipcs_rw != NULL) {
-        pcmk__drop_all_clients(ipcs_rw);
-        g_clear_pointer(&ipcs_rw, qb_ipcs_destroy);
-    }
+    pcmk__drop_all_clients(ipcs_rw);
+    g_clear_pointer(&ipcs_rw, qb_ipcs_destroy);
 
-    if (ipcs_shm != NULL) {
-        pcmk__drop_all_clients(ipcs_shm);
-        g_clear_pointer(&ipcs_shm, qb_ipcs_destroy);
-    }
+    pcmk__drop_all_clients(ipcs_shm);
+    g_clear_pointer(&ipcs_shm, qb_ipcs_destroy);
 
     based_drop_remote_clients();