Fix races between irc_send_raw() and normal prpl-internal sends.

Sat, 25 Jun 2011 02:42:08 +0000

author
Ethan Blanton <elb@pidgin.im>
date
Sat, 25 Jun 2011 02:42:08 +0000
changeset 31929
3c1f2bba1933
parent 31927
2a3e0c6e8304
child 31930
7e061aebed8b

Fix races between irc_send_raw() and normal prpl-internal sends.

Fixes #14263

libpurple/protocols/irc/irc.c file | annotate | diff | comparison | revisions
libpurple/protocols/irc/irc.h file | annotate | diff | comparison | revisions
--- a/libpurple/protocols/irc/irc.c	Fri Jun 24 16:44:06 2011 +0000
+++ b/libpurple/protocols/irc/irc.c	Sat Jun 25 02:42:08 2011 +0000
@@ -101,7 +101,11 @@
 static int irc_send_raw(PurpleConnection *gc, const char *buf, int len)
 {
 	struct irc_conn *irc = (struct irc_conn*)gc->proto_data;
-	return do_send(irc, buf, len);
+	if (len == -1) {
+		len = strlen(buf);
+	}
+	irc_send_len(irc, buf, len);
+	return len;
 }
 
 static void
@@ -144,16 +148,18 @@
 
 int irc_send(struct irc_conn *irc, const char *buf)
 {
-	int ret, buflen;
+    return irc_send_len(irc, buf, strlen(buf));
+}
+
+int irc_send_len(struct irc_conn *irc, const char *buf, int buflen)
+{
+	int ret;
  	char *tosend= g_strdup(buf);
 
 	purple_signal_emit(_irc_plugin, "irc-sending-text", purple_account_get_connection(irc->account), &tosend);
 	if (tosend == NULL)
 		return 0;
 
-	buflen = strlen(tosend);
-
-
 	/* If we're not buffering writes, try to send immediately */
 	if (!irc->writeh)
 		ret = do_send(irc, tosend, buflen);
--- a/libpurple/protocols/irc/irc.h	Fri Jun 24 16:44:06 2011 +0000
+++ b/libpurple/protocols/irc/irc.h	Sat Jun 25 02:42:08 2011 +0000
@@ -106,6 +106,7 @@
 typedef int (*IRCCmdCallback) (struct irc_conn *irc, const char *cmd, const char *target, const char **args);
 
 int irc_send(struct irc_conn *irc, const char *buf);
+int irc_send_len(struct irc_conn *irc, const char *buf, int len);
 gboolean irc_blist_timeout(struct irc_conn *irc);
 gboolean irc_who_channel_timeout(struct irc_conn *irc);
 void irc_buddy_query(struct irc_conn *irc);

mercurial