rpms/gvfs/F-9 gvfs-ftp-read-directory-2.patch, NONE, 1.1 gvfs.spec, 1.54, 1.55
Tomas Bzatek (tbzatek)
fedora-extras-commits at redhat.com
Tue Jun 10 13:58:56 UTC 2008
- Previous message: rpms/man-pages/devel man-pages-2.80-malloc_h.patch, NONE, 1.1 .cvsignore, 1.49, 1.50 man-pages.spec, 1.97, 1.98 sources, 1.48, 1.49
- Next message: rpms/kdelibs/F-9 kdelibs-4.0.5-kio-ftp-overwrite.patch, NONE, 1.1 kdelibs-4.0.5-kio-subjob.patch, NONE, 1.1 kdelibs-4.0.5-kprotocolcombo.patch, NONE, 1.1 kdelibs-4.0.5-strigi-metainfo.patch, NONE, 1.1 kdelibs.spec, 1.324, 1.325
- Messages sorted by:
[ date ]
[ thread ]
[ subject ]
[ author ]
Author: tbzatek
Update of /cvs/extras/rpms/gvfs/F-9
In directory cvs-int.fedora.redhat.com:/tmp/cvs-serv9246
Modified Files:
gvfs.spec
Added Files:
gvfs-ftp-read-directory-2.patch
Log Message:
* Tue Jun 10 2008 Tomas Bzatek <tbzatek at redhat.com> - 0.2.4-3
- Fix transfer of whole directories from FTP (#448560)
gvfs-ftp-read-directory-2.patch:
--- NEW FILE gvfs-ftp-read-directory-2.patch ---
Index: daemon/gvfsbackendftp.c
===================================================================
--- daemon/gvfsbackendftp.c (revision 1801)
+++ daemon/gvfsbackendftp.c (working copy)
@@ -134,6 +134,8 @@
#define STATUS_GROUP(status) ((status) / 100)
+typedef void (* Ftp550Handler) (FtpConnection *conn, const FtpFile *file);
+
/*** FTP CONNECTION ***/
struct _FtpConnection
@@ -249,8 +251,8 @@
case 550: /* Requested action not taken. File unavailable (e.g., file not found, no access). */
/* FIXME: This is a lot of different errors. So we have to pretend to
* be smart here. */
- code = G_IO_ERROR_NOT_FOUND;
- msg = _("File unavailable");
+ code = G_IO_ERROR_FAILED;
+ msg = _("Operation failed");
break;
case 451: /* Requested action aborted: local error in processing. */
code = G_IO_ERROR_FAILED;
@@ -297,6 +299,7 @@
* RESPONSE_PASS_300: Don't treat 3XX responses, but return them
* RESPONSE_PASS_400: Don't treat 4XX responses, but return them
* RESPONSE_PASS_500: Don't treat 5XX responses, but return them
+ * RESPONSE_PASS_550: Don't treat 550 responses, but return them
* RESPONSE_FAIL_200: Fail on a 2XX response
*/
@@ -305,7 +308,8 @@
RESPONSE_PASS_300 = (1 << 1),
RESPONSE_PASS_400 = (1 << 2),
RESPONSE_PASS_500 = (1 << 3),
- RESPONSE_FAIL_200 = (1 << 4)
+ RESPONSE_PASS_550 = (1 << 4),
+ RESPONSE_FAIL_200 = (1 << 5)
} ResponseFlags;
/**
@@ -459,7 +463,7 @@
return 0;
break;
case 5:
- if (flags & RESPONSE_PASS_500)
+ if ((flags & RESPONSE_PASS_500) || (response == 550 && (flags & RESPONSE_PASS_550)))
break;
ftp_connection_set_error_from_response (conn, response);
return 0;
@@ -564,6 +568,55 @@
}
static void
+ftp_connection_check_file (FtpConnection *conn,
+ const Ftp550Handler *handlers,
+ const FtpFile *file)
+{
+ while (*handlers && !ftp_connection_in_error (conn))
+ {
+ (*handlers) (conn, file);
+ handlers++;
+ }
+}
+
+static guint
+ftp_connection_send_and_check (FtpConnection *conn,
+ ResponseFlags flags,
+ const Ftp550Handler *handlers,
+ const FtpFile *file,
+ const char *format,
+ ...) G_GNUC_PRINTF (5, 6);
+static guint
+ftp_connection_send_and_check (FtpConnection *conn,
+ ResponseFlags flags,
+ const Ftp550Handler *handlers,
+ const FtpFile *file,
+ const char *format,
+ ...)
+{
+ va_list varargs;
+ guint response;
+
+ /* check that there's no 550 handling used - don't allow bad use of API */
+ g_return_val_if_fail ((flags & RESPONSE_PASS_550) == 0, 0);
+ g_return_val_if_fail (handlers != NULL, 0);
+ g_return_val_if_fail (file != NULL, 0);
+
+ va_start (varargs, format);
+ response = ftp_connection_sendv (conn,
+ flags | RESPONSE_PASS_550,
+ format,
+ varargs);
+ va_end (varargs);
+ if (response == 550)
+ {
+ ftp_connection_check_file (conn, handlers, file);
+ response = 0;
+ }
+ return response;
+}
+
+static void
ftp_connection_parse_features (FtpConnection *conn)
{
struct {
@@ -1461,6 +1514,21 @@
}
static void
+error_550_is_directory (FtpConnection *conn, const FtpFile *file)
+{
+ guint response = ftp_connection_send (conn,
+ RESPONSE_PASS_550,
+ "CWD %s", file);
+
+ if (response == 250)
+ {
+ g_set_error (&conn->error, G_IO_ERROR,
+ G_IO_ERROR_IS_DIRECTORY,
+ _("File is a directory"));
+ }
+}
+
+static void
do_open_for_read (GVfsBackend *backend,
GVfsJobOpenForRead *job,
const char *filename)
@@ -1468,6 +1536,7 @@
GVfsBackendFtp *ftp = G_VFS_BACKEND_FTP (backend);
FtpConnection *conn;
FtpFile *file;
+ static const Ftp550Handler open_read_handlers[] = { error_550_is_directory, NULL };
conn = g_vfs_backend_ftp_pop_connection (ftp, G_VFS_JOB (job));
if (!conn)
@@ -1476,9 +1545,11 @@
ftp_connection_ensure_data_connection (conn);
file = ftp_filename_from_gvfs_path (conn, filename);
- ftp_connection_send (conn,
- RESPONSE_PASS_100 | RESPONSE_FAIL_200,
- "RETR %s", file);
+ ftp_connection_send_and_check (conn,
+ RESPONSE_PASS_100 | RESPONSE_FAIL_200,
+ &open_read_handlers[0],
+ file,
+ "RETR %s", file);
g_free (file);
if (ftp_connection_in_error (conn))
Index: gvfs.spec
===================================================================
RCS file: /cvs/extras/rpms/gvfs/F-9/gvfs.spec,v
retrieving revision 1.54
retrieving revision 1.55
diff -u -r1.54 -r1.55
--- gvfs.spec 9 Jun 2008 15:21:01 -0000 1.54
+++ gvfs.spec 10 Jun 2008 13:58:15 -0000 1.55
@@ -1,7 +1,7 @@
Summary: Backends for the gio framework in GLib
Name: gvfs
Version: 0.2.4
-Release: 2%{?dist}
+Release: 3%{?dist}
License: LGPLv2+
Group: System Environment/Libraries
URL: http://www.gtk.org
@@ -44,6 +44,10 @@
# https://bugzilla.redhat.com/show_bug.cgi?id=450453
Patch5: gvfs-obexftp-fallback-icon.patch
+# http://bugzilla.gnome.org/show_bug.cgi?id=522933
+Patch6: gvfs-ftp-read-directory-2.patch
+
+
%description
The gvfs package provides backend implementations for the gio
framework in GLib. It includes ftp, sftp, cifs.
@@ -76,6 +80,7 @@
%patch3 -p0 -b .archive-iso
%patch4 -p1 -b .debug
%patch5 -p0 -b .obexftp
+%patch6 -p0 -b .recurse
%build
@@ -191,6 +196,9 @@
%changelog
+* Tue Jun 10 2008 Tomas Bzatek <tbzatek at redhat.com> - 0.2.4-3
+- Fix transfer of whole directories from FTP (#448560)
+
* Mon Jun 9 2008 Tomas Bzatek <tbzatek at redhat.com> - 0.2.4-2
- obexftp: Fix the ugly icons when the mime-type of a file
isn't known to us
- Previous message: rpms/man-pages/devel man-pages-2.80-malloc_h.patch, NONE, 1.1 .cvsignore, 1.49, 1.50 man-pages.spec, 1.97, 1.98 sources, 1.48, 1.49
- Next message: rpms/kdelibs/F-9 kdelibs-4.0.5-kio-ftp-overwrite.patch, NONE, 1.1 kdelibs-4.0.5-kio-subjob.patch, NONE, 1.1 kdelibs-4.0.5-kprotocolcombo.patch, NONE, 1.1 kdelibs-4.0.5-strigi-metainfo.patch, NONE, 1.1 kdelibs.spec, 1.324, 1.325
- Messages sorted by:
[ date ]
[ thread ]
[ subject ]
[ author ]
More information about the scm-commits
mailing list