/soc/2013/ankitkv/gobjectification: aa8cf5f66b2c: Merged soc.201...
Ankit Vani
a at nevitus.org
Thu Aug 29 05:10:42 EDT 2013
Changeset: aa8cf5f66b2c4a34f3615eb652137beaf618628a
Author: Ankit Vani <a at nevitus.org>
Date: 2013-08-29 14:40 +0530
Branch: soc.2013.gobjectification.plugins
URL: https://hg.pidgin.im/soc/2013/ankitkv/gobjectification/rev/aa8cf5f66b2c
Description:
Merged soc.2013.gobjectification branch
diffstat:
libpurple/Makefile.am | 2 +
libpurple/Makefile.mingw | 1 +
libpurple/http.c | 185 ++++-------------------
libpurple/purple-socket.c | 354 ++++++++++++++++++++++++++++++++++++++++++++++
libpurple/purple-socket.h | 200 +++++++++++++++++++++++++
5 files changed, 594 insertions(+), 148 deletions(-)
diffs (truncated from 925 to 300 lines):
diff --git a/libpurple/Makefile.am b/libpurple/Makefile.am
--- a/libpurple/Makefile.am
+++ b/libpurple/Makefile.am
@@ -83,6 +83,7 @@ purple_coresources = \
proxy.c \
protocol.c \
protocols.c \
+ purple-socket.c \
request.c \
roomlist.c \
savedstatuses.c \
@@ -155,6 +156,7 @@ purple_coreheaders = \
proxy.h \
protocol.h \
protocols.h \
+ purple-socket.h \
request.h \
roomlist.h \
savedstatuses.h \
diff --git a/libpurple/Makefile.mingw b/libpurple/Makefile.mingw
--- a/libpurple/Makefile.mingw
+++ b/libpurple/Makefile.mingw
@@ -109,6 +109,7 @@ C_SRC = \
privacy.c \
proxy.c \
prpl.c \
+ purple-socket.c \
request.c \
roomlist.c \
savedstatuses.c \
diff --git a/libpurple/http.c b/libpurple/http.c
--- a/libpurple/http.c
+++ b/libpurple/http.c
@@ -31,6 +31,7 @@
#include "debug.h"
#include "ntlm.h"
+#include "purple-socket.h"
#include <zlib.h>
@@ -56,22 +57,13 @@ typedef struct _PurpleHttpKeepaliveReque
typedef struct _PurpleHttpGzStream PurpleHttpGzStream;
-typedef void (*PurpleHttpSocketConnectCb)(PurpleHttpSocket *hs,
- const gchar *error, gpointer user_data);
-
struct _PurpleHttpSocket
{
- gboolean is_ssl;
+ PurpleSocket *ps;
+
gboolean is_busy;
guint use_count;
PurpleHttpKeepaliveHost *host;
-
- PurpleSslConnection *ssl_connection;
- PurpleProxyConnectData *raw_connection;
- int fd;
- guint inpa;
- PurpleHttpSocketConnectCb connect_cb;
- gpointer cb_data;
};
struct _PurpleHttpRequest
@@ -183,7 +175,7 @@ struct _PurpleHttpCookieJar
struct _PurpleHttpKeepaliveRequest
{
PurpleConnection *gc;
- PurpleHttpSocketConnectCb cb;
+ PurpleSocketConnectCb cb;
gpointer user_data;
PurpleHttpKeepaliveHost *host;
@@ -255,7 +247,7 @@ gchar * purple_http_cookie_jar_dump(Purp
static PurpleHttpKeepaliveRequest *
purple_http_keepalive_pool_request(PurpleHttpKeepalivePool *pool,
PurpleConnection *gc, const gchar *host, int port, gboolean is_ssl,
- PurpleHttpSocketConnectCb cb, gpointer user_data);
+ PurpleSocketConnectCb cb, gpointer user_data);
static void
purple_http_keepalive_pool_request_cancel(PurpleHttpKeepaliveRequest *req);
static void
@@ -463,43 +455,6 @@ purple_http_gz_free(PurpleHttpGzStream *
/*** HTTP Sockets *************************************************************/
-static void _purple_http_socket_connected_raw(gpointer _hs, gint fd,
- const gchar *error_message)
-{
- PurpleHttpSocket *hs = _hs;
-
- hs->raw_connection = NULL;
-
- if (fd == -1 || error_message != NULL) {
- if (error_message == NULL)
- error_message = _("Unknown error");
- hs->connect_cb(hs, error_message, hs->cb_data);
- return;
- }
-
- hs->fd = fd;
- hs->connect_cb(hs, NULL, hs->cb_data);
-}
-
-static void _purple_http_socket_connected_ssl(gpointer _hs,
- PurpleSslConnection *ssl_connection, PurpleInputCondition cond)
-{
- PurpleHttpSocket *hs = _hs;
-
- hs->fd = hs->ssl_connection->fd;
- hs->connect_cb(hs, NULL, hs->cb_data);
-}
-
-static void _purple_http_socket_connected_ssl_error(
- PurpleSslConnection *ssl_connection, PurpleSslErrorType error,
- gpointer _hs)
-{
- PurpleHttpSocket *hs = _hs;
-
- hs->ssl_connection = NULL;
- hs->connect_cb(hs, purple_ssl_strerror(error), hs->cb_data);
-}
-
static gchar *
purple_http_socket_hash(const gchar *host, int port, gboolean is_ssl)
{
@@ -507,37 +462,18 @@ purple_http_socket_hash(const gchar *hos
}
static PurpleHttpSocket *
-purple_http_socket_connect_new(PurpleConnection *gc, const gchar *host, int port, gboolean is_ssl, PurpleHttpSocketConnectCb cb, gpointer user_data)
+purple_http_socket_connect_new(PurpleConnection *gc, const gchar *host,
+ int port, gboolean is_ssl, PurpleSocketConnectCb cb, gpointer user_data)
{
PurpleHttpSocket *hs = g_new0(PurpleHttpSocket, 1);
- PurpleAccount *account = NULL;
-
- if (gc != NULL)
- account = purple_connection_get_account(gc);
-
- hs->is_ssl = is_ssl;
- hs->connect_cb = cb;
- hs->cb_data = user_data;
- hs->fd = -1;
-
- if (is_ssl) {
- if (!purple_ssl_is_supported()) {
- g_free(hs);
- return NULL;
- }
-
- hs->ssl_connection = purple_ssl_connect(account,
- host, port,
- _purple_http_socket_connected_ssl,
- _purple_http_socket_connected_ssl_error, hs);
- } else {
- hs->raw_connection = purple_proxy_connect(gc, account,
- host, port,
- _purple_http_socket_connected_raw, hs);
- }
-
- if (hs->ssl_connection == NULL &&
- hs->raw_connection == NULL) {
+
+ hs->ps = purple_socket_new(gc);
+ purple_socket_set_data(hs->ps, "hs", hs);
+ purple_socket_set_tls(hs->ps, is_ssl);
+ purple_socket_set_host(hs->ps, host);
+ purple_socket_set_port(hs->ps, port);
+ if (!purple_socket_connect(hs->ps, cb, user_data)) {
+ purple_socket_destroy(hs->ps);
g_free(hs);
return NULL;
}
@@ -548,50 +484,6 @@ purple_http_socket_connect_new(PurpleCon
return hs;
}
-static int
-purple_http_socket_read(PurpleHttpSocket *hs, gchar *buf, size_t len)
-{
- g_return_val_if_fail(hs != NULL, -1);
- g_return_val_if_fail(buf != NULL, -1);
-
- if (hs->is_ssl)
- return purple_ssl_read(hs->ssl_connection, buf, sizeof(buf));
- else
- return read(hs->fd, buf, sizeof(buf));
-}
-
-static int
-purple_http_socket_write(PurpleHttpSocket *hs, const gchar *buf, size_t len)
-{
- g_return_val_if_fail(hs != NULL, -1);
- g_return_val_if_fail(buf != NULL, -1);
-
- if (hs->is_ssl)
- return purple_ssl_write(hs->ssl_connection, buf, len);
- else
- return write(hs->fd, buf, len);
-}
-
-static void
-purple_http_socket_dontwatch(PurpleHttpSocket *hs)
-{
- g_return_if_fail(hs != NULL);
-
- if (hs->inpa > 0)
- purple_input_remove(hs->inpa);
- hs->inpa = 0;
-}
-
-static void
-purple_http_socket_watch(PurpleHttpSocket *hs, PurpleInputCondition cond,
- PurpleInputFunction func, gpointer user_data)
-{
- g_return_if_fail(hs != NULL);
-
- purple_http_socket_dontwatch(hs);
- hs->inpa = purple_input_add(hs->fd, cond, func, user_data);
-}
-
static void
purple_http_socket_close_free(PurpleHttpSocket *hs)
{
@@ -601,19 +493,7 @@ purple_http_socket_close_free(PurpleHttp
if (purple_debug_is_verbose())
purple_debug_misc("http", "destroying socket: %p\n", hs);
- if (hs->inpa != 0)
- purple_input_remove(hs->inpa);
-
- if (hs->is_ssl) {
- if (hs->ssl_connection != NULL)
- purple_ssl_close(hs->ssl_connection);
- } else {
- if (hs->raw_connection != NULL)
- purple_proxy_connect_cancel(hs->raw_connection);
- if (hs->fd > 0)
- close(hs->fd);
- }
-
+ purple_socket_destroy(hs->ps);
g_free(hs);
}
@@ -1199,7 +1079,7 @@ static gboolean _purple_http_recv_loopbo
gchar buf[4096];
gboolean got_anything;
- len = purple_http_socket_read(hc->socket, buf, sizeof(buf));
+ len = purple_socket_read(hc->socket->ps, (guchar*)buf, sizeof(buf));
got_anything = (len > 0);
if (len < 0 && errno == EAGAIN)
@@ -1462,8 +1342,8 @@ static void _purple_http_send(gpointer _
purple_debug_warning("http", "Nothing to write\n");
written = 0;
} else {
- written = purple_http_socket_write(hc->socket, write_from,
- write_len);
+ written = purple_socket_write(hc->socket->ps,
+ (const guchar*)write_from, write_len);
}
if (written < 0 && errno == EAGAIN)
@@ -1506,7 +1386,7 @@ static void _purple_http_send(gpointer _
/* request is completely written, let's read the response */
hc->is_reading = TRUE;
- purple_http_socket_watch(hc->socket, PURPLE_INPUT_READ,
+ purple_socket_watch(hc->socket->ps, PURPLE_INPUT_READ,
_purple_http_recv, hc);
}
@@ -1531,10 +1411,15 @@ static void _purple_http_disconnect(Purp
}
}
-static void _purple_http_connected(PurpleHttpSocket *hs, const gchar *error, gpointer _hc)
+static void
+_purple_http_connected(PurpleSocket *ps, const gchar *error, gpointer _hc)
{
+ PurpleHttpSocket *hs = NULL;
PurpleHttpConnection *hc = _hc;
+ if (ps != NULL)
+ hs = purple_socket_get_data(ps, "hs");
+
hc->socket_request = NULL;
hc->socket = hs;
@@ -1544,7 +1429,7 @@ static void _purple_http_connected(Purp
return;
}
- purple_http_socket_watch(hs, PURPLE_INPUT_WRITE, _purple_http_send, hc);
+ purple_socket_watch(hs->ps, PURPLE_INPUT_WRITE, _purple_http_send, hc);
More information about the Commits
mailing list