Refactor: libstonithd: Drop null_field Coverity suppression

Coverity was flagging this as an error because it didn't know whether
pcmk__trace() might set entry->notify to NULL. Coverity doesn't have a
model for qb_log_from_external_source(), which pcmk__trace() calls.

We could provide a model, but instead we just drop the "%p", which
doesn't seem helpful.

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

Author: nrwahl2

lib/fencing/st_client.c

@@ -1560,14 +1560,14 @@ stonith_send_notification(gpointer data, gpointer user_data)
         return;
 
     } else if (!pcmk__str_eq(entry->event, event, pcmk__str_none)) {
-        pcmk__trace("Skipping callback - event mismatch %p/%s vs. %s", entry, entry->event, event);
+        pcmk__trace("Skipping callback - event mismatch %s vs. %s",
+                    entry->event, event);
         return;
     }
 
     st_event = xml_to_event(blob->xml);
 
-    pcmk__trace("Invoking callback for %p/%s event...", entry, event);
-    // coverity[null_field]
+    pcmk__trace("Invoking callback for %s event...", event);
     entry->notify(blob->stonith, st_event);
     pcmk__trace("Callback invoked...");