From 24aefb5b3d27cc7c763c8b386735f1c95ac22a90 Mon Sep 17 00:00:00 2001
From: defanor <defanor@uberspace.net>
Date: Sat, 30 May 2020 11:36:58 +0300
Subject: Remove callback functions from rexmpp_init arguments

Leaving required parameters only (i.e., just an initial JID).
---
 src/rexmpp.c | 25 +++++++++++++------------
 1 file changed, 13 insertions(+), 12 deletions(-)

(limited to 'src/rexmpp.c')

diff --git a/src/rexmpp.c b/src/rexmpp.c
index c74b92b..959626a 100644
--- a/src/rexmpp.c
+++ b/src/rexmpp.c
@@ -203,18 +203,13 @@ xmlNodePtr rexmpp_xml_default_disco_info () {
 
 int rexmpp_sasl_cb (Gsasl *ctx, Gsasl_session *sctx, Gsasl_property prop) {
   rexmpp_t *s = gsasl_callback_hook_get(ctx);
-  if (s == NULL) {
+  if (s == NULL || s->sasl_property_cb == NULL) {
     return GSASL_NO_CALLBACK;
   }
   return s->sasl_property_cb(s, prop);
 }
 
-rexmpp_err_t rexmpp_init (rexmpp_t *s,
-                          const char *jid,
-                          log_function_t log_function,
-                          sasl_property_cb_t sasl_property_cb,
-                          xml_in_cb_t xml_in_cb,
-                          xml_out_cb_t xml_out_cb)
+rexmpp_err_t rexmpp_init (rexmpp_t *s, const char *jid)
 {
   int err;
   xmlSAXHandler sax = {
@@ -262,21 +257,23 @@ rexmpp_err_t rexmpp_init (rexmpp_t *s,
   s->reconnect_number = 0;
   s->next_reconnect_time.tv_sec = 0;
   s->next_reconnect_time.tv_usec = 0;
-  s->initial_jid = jid;
+  s->initial_jid = NULL;
   s->assigned_jid = NULL;
   s->stanza_queue_size = 1024;
   s->send_queue_size = 1024;
   s->iq_queue_size = 1024;
-  s->log_function = log_function;
-  s->sasl_property_cb = sasl_property_cb;
-  s->xml_in_cb = xml_in_cb;
-  s->xml_out_cb = xml_out_cb;
+  s->log_function = NULL;
+  s->sasl_property_cb = NULL;
+  s->xml_in_cb = NULL;
+  s->xml_out_cb = NULL;
 
   if (jid == NULL) {
     rexmpp_log(s, LOG_CRIT, "No initial JID is provided.");
     return REXMPP_E_JID;
   }
 
+  s->initial_jid = strdup(jid);
+
   s->xml_parser = xmlCreatePushParserCtxt(&sax, s, "", 0, NULL);
 
   if (s->xml_parser == NULL) {
@@ -402,6 +399,10 @@ void rexmpp_done (rexmpp_t *s) {
   ares_destroy(s->resolver_channel);
   ares_library_cleanup();
   xmlFreeParserCtxt(s->xml_parser);
+  if (s->initial_jid != NULL) {
+    free(s->initial_jid);
+    s->initial_jid = NULL;
+  }
   if (s->stream_id != NULL) {
     free(s->stream_id);
     s->stream_id = NULL;
-- 
cgit v1.2.3