pidgin: libpurple/ft.c comparison

comparison libpurple/ft.c @ 29344:10eaca0bf01a

propagate from branch 'im.pidgin.pidgin' (head 8f0a3e68547fbc66188357edbf6694bd8dd2d557) to branch 'im.pidgin.pidgin.next.minor' (head 63bbded7257d603ff4c002350caa652f85d2ff9e)

author	John Bailey <rekkanoryo@rekkanoryo.org>
date	Fri, 25 Dec 2009 22:08:46 +0000
parents	1e19c439fa7a 16dfa8df7dd1
children	ba8577b87d2e

comparison

equal deleted inserted replaced

-:bf4a33bd020b
+:10eaca0bf01a
 	if (ui_ops != NULL && ui_ops->new_xfer != NULL)
 		ui_ops->new_xfer(xfer);
 	xfers = g_list_prepend(xfers, xfer);
+	if (purple_debug_is_verbose())
+		purple_debug_info("xfer", "new %p [%d]\n", xfer, xfer->ref);
 	return xfer;
 }
 static void
 purple_xfer_destroy(PurpleXfer *xfer)
 {
 	PurpleXferUiOps *ui_ops;
 	PurpleXferPrivData *priv;
 	g_return_if_fail(xfer != NULL);
-	priv = g_hash_table_lookup(xfers_data, xfer);
 	/* Close the file browser, if it's open */
 	purple_request_close_with_handle(xfer);
 	if (purple_xfer_get_status(xfer) == PURPLE_XFER_STATUS_STARTED)
 purple_xfer_ref(PurpleXfer *xfer)
 {
 	g_return_if_fail(xfer != NULL);
 	xfer->ref++;
+	if (purple_debug_is_verbose())
+		purple_debug_info("xfer", "ref'd %p [%d]\n", xfer, xfer->ref);
 }
 void
 purple_xfer_unref(PurpleXfer *xfer)
 {
 	g_return_if_fail(xfer != NULL);
 	g_return_if_fail(xfer->ref > 0);
 	xfer->ref--;
+	if (purple_debug_is_verbose())
+		purple_debug_info("xfer", "unref'd %p [%d]\n", xfer, xfer->ref);
 	if (xfer->ref == 0)
 		purple_xfer_destroy(xfer);
 }
 static void
 purple_xfer_choose_file_ok_cb(void *user_data, const char *filename)
 {
 	PurpleXfer *xfer;
+	PurpleXferType type;
 	struct stat st;
 	gchar *dir;
 	xfer = (PurpleXfer *)user_data;
+	type = purple_xfer_get_type(xfer);
 	if (g_stat(filename, &st) != 0) {
 		/* File not found. */
-		if (purple_xfer_get_type(xfer) == PURPLE_XFER_RECEIVE) {
+		if (type == PURPLE_XFER_RECEIVE) {
 #ifndef _WIN32
 			int mode = W_OK;
 #else
 			int mode = F_OK;
 #endif
 			g_free(dir);
 		}
 		else {
 			purple_xfer_show_file_error(xfer, filename);
-			purple_xfer_request_denied(xfer);
+			purple_xfer_cancel_local(xfer);
 		}
 	}
-	else if ((purple_xfer_get_type(xfer) == PURPLE_XFER_SEND) &&
+	else if ((type == PURPLE_XFER_SEND) && (st.st_size == 0)) {
-			 (st.st_size == 0)) {
 		purple_notify_error(NULL, NULL,
 						  _("Cannot send a file of 0 bytes."), NULL);
-		purple_xfer_request_denied(xfer);
+		purple_xfer_cancel_local(xfer);
 	}
-	else if ((purple_xfer_get_type(xfer) == PURPLE_XFER_SEND) &&
+	else if ((type == PURPLE_XFER_SEND) && S_ISDIR(st.st_mode)) {
-			 S_ISDIR(st.st_mode)) {
 		/*
 		 * XXX - Sending a directory should be valid for some protocols.
 		 */
 		purple_notify_error(NULL, NULL,
 						  _("Cannot send a directory."), NULL);
-		purple_xfer_request_denied(xfer);
+		purple_xfer_cancel_local(xfer);
 	}
-	else if ((purple_xfer_get_type(xfer) == PURPLE_XFER_RECEIVE) &&
+	else if ((type == PURPLE_XFER_RECEIVE) && S_ISDIR(st.st_mode)) {
-			 S_ISDIR(st.st_mode)) {
 		char *msg, *utf8;
 		utf8 = g_filename_to_utf8(filename, -1, NULL, NULL, NULL);
 		msg = g_strdup_printf(
 					_("%s is not a regular file. Cowardly refusing to overwrite it.\n"), utf8);
 		g_free(utf8);
 		purple_notify_error(NULL, NULL, msg, NULL);
 		g_free(msg);
 		purple_xfer_request_denied(xfer);
 	}
+	else if (type == PURPLE_XFER_SEND) {
+#ifndef _WIN32
+		int mode = R_OK;
+#else
+		int mode = F_OK;
+#endif
+		if (g_access(filename, mode) == 0) {
+			purple_xfer_request_accepted(xfer, filename);
+		} else {
+			purple_xfer_ref(xfer);
+			purple_notify_message(
+				NULL, PURPLE_NOTIFY_MSG_ERROR, NULL,
+				_("File is not readable."), NULL,
+				(PurpleNotifyCloseCallback)purple_xfer_choose_file, xfer);
+		}
+	}
 	else {
 		purple_xfer_request_accepted(xfer, filename);
 	}
 	purple_xfer_unref(xfer);
 purple_xfer_choose_file_cancel_cb(void *user_data, const char *filename)
 {
 	PurpleXfer *xfer = (PurpleXfer *)user_data;
 	purple_xfer_set_status(xfer, PURPLE_XFER_STATUS_CANCEL_LOCAL);
-	purple_xfer_request_denied(xfer);
+	if (purple_xfer_get_type(xfer) == PURPLE_XFER_SEND)
+		purple_xfer_cancel_local(xfer);
+	else
+		purple_xfer_request_denied(xfer);
+	purple_xfer_unref(xfer);
 }
 static int
 purple_xfer_choose_file(PurpleXfer *xfer)
 {
 		r = (xfer->ops.write)(buffer, s, xfer);
 	} else {
 		r = write(xfer->fd, buffer, s);
 		if (r < 0 && errno == EAGAIN)
 			r = 0;
+		if ((purple_xfer_get_bytes_sent(xfer)+r) >= purple_xfer_get_size(xfer))
+			purple_xfer_set_completed(xfer, TRUE);
 	}
 	return r;
 }
 				purple_xfer_cancel_local(xfer);
 				g_free(buffer);
 				return;
 			}
+			if ((purple_xfer_get_size(xfer) > 0) &&
+				((purple_xfer_get_bytes_sent(xfer)+r) >= purple_xfer_get_size(xfer)))
+				purple_xfer_set_completed(xfer, TRUE);
 		} else if(r < 0) {
 			purple_xfer_cancel_remote(xfer);
 			g_free(buffer);
 			return;
 		}
 				if (xfer->watcher != 0) {
 					purple_input_remove(xfer->watcher);
 					xfer->watcher = 0;
 				}
 				/*
-				 * if we requested 0 bytes it's only normal that en up here
+				 * if we requested 0 bytes it's only normal that end up here
 				 * we shouldn't return as we still have something to
 				 * write in priv->buffer
 				 */
 				if (s != 0)
 					return;

Mercurial > pidgin

comparison libpurple/ft.c @ 29344:10eaca0bf01a