Incorrect handling of FTP 'pwd' reply resulting in nautilus mount errors

Bug #551934 reported by Dmitry Pankratov
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
gvfs
Fix Released
Medium
gvfs (Ubuntu)
Fix Released
Low
Unassigned

Bug Description

Binary package hint: gvfs

When GVFS sends 'pwd' command to FTP server is gets reply in the following format:
257 "/" is current directory.

This line is not parsed correctly and everything following the "257" code is taken as a default location resulting in erroneous behavior of Nautilus.

Tested in Ubuntu 10.04 AMD64, GVFS version 1.6.0-0ubuntu1.

Revision history for this message
Dmitry Pankratov (dremon) wrote :

Attached is my proposed patch for this bug.

Revision history for this message
Sebastien Bacher (seb128) wrote :

Thank you for your bug report. The issue is an upstream one and it would be nice if somebody having it could send the bug the to the people writting the software (https://wiki.ubuntu.com/Bugs/Upstream/GNOME)

Changed in gvfs (Ubuntu):
importance: Undecided → Low
Revision history for this message
Josh Holland (jshholland) wrote :

Forwarded upstream.

Revision history for this message
Josh Holland (jshholland) wrote :

I'm happy to apply the patch to the Lucid package too, if required.

Revision history for this message
Sebastien Bacher (seb128) wrote :

Thank you for sending the bug to GNOME, let's see if they comment otherwise we can consider that as a lucid change

Changed in gvfs (Ubuntu):
status: New → Triaged
Revision history for this message
Dmitry Pankratov (dremon) wrote :

The bug has been fixed upstream.

Revision history for this message
Sebastien Bacher (seb128) wrote :

the change will be in lucid with the next update

Changed in gvfs (Ubuntu):
status: Triaged → Fix Committed
Revision history for this message
Sebastien Bacher (seb128) wrote :

gvfs (1.6.0+git20100414-0ubuntu1) lucid; urgency=low

  * New upstream git snapshot (since GNOME 2.30.1 will be too late for lucid):
    - afc: Detect iPad and set correct icon name
    - ftp: Parse PWD reply according to RFC
    - ftp: Add missing \n to debug message
    - Attach cancellable to async result and check in finish functions
      (LP: #544306)
    - Disable x-content-types for jailbroken iDevices
    - Translation updates.
  * Drop 04_dont_automount_internal_partitions.patch, it was fixed upstream a
    while ago with a different approach.
  * 02_music-player-mimetype.patch: Update to current patch tagging format.
  * debian/control.in: Bump Standards-Version to 3.8.4 (no changes necessary).
  * Add 01_dont_ignore_blank_cd_drives.patch: Extend the special casing of CD
    drives in should_drive_be_ignored() to also cover blank media. Otherwise
    we lose the drive for the blank media volume, and with it the possibility
    to eject it. (LP: #425104)

Changed in gvfs (Ubuntu):
status: Fix Committed → Fix Released
Changed in gvfs:
status: Unknown → Fix Released
Changed in gvfs:
importance: Unknown → Medium
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.