custom smb port changes in pcmanfm's address bar

Bug #1578453 reported by Joonhee, Jun
12
This bug affects 2 people
Affects Status Importance Assigned to Milestone
pcmanfm (Ubuntu)
Confirmed
Undecided
Unassigned

Bug Description

Hello there,

Thank you so much for all your efforts to develop such a great OS & applications suite.

I run a samba file sharing server in my intranet. (Server's TCP listening port is 139)
And I connect to the server over SSH tunneling from outside. (Localhost's TCP listening port for SSH tunneling is 50139)

When I try to connect smb://localhost:50139, PCManFM lists shared directories well.

BUT, once I double-click one of the list, the URI in address bar loses ":50139", my customized TCP port.

ex.
smb://localhost:50139 ---> I choose a shared directory, "A" ---> smb://localhost/A

PCManFM tells an error "Failed to mount Windows share: Connection refused" naturally.
And it connects well after I modify the URL to smb://localhost:50139/A in address bar.

I would like to inform you this as I found this bug just a minute ago.
I use Lubuntu 16.04 LTS, and "about" shows the versions are

PCManFM 1.2.4,
Lightweight file manager using LibFM ver. 1.2.4

I will also report this bug to LXDE community.
Thank you!

June

ProblemType: Bug
DistroRelease: Ubuntu 16.04
Package: pcmanfm 1.2.4-1
ProcVersionSignature: Ubuntu 4.4.0-21.37-generic 4.4.6
Uname: Linux 4.4.0-21-generic x86_64
ApportVersion: 2.20.1-0ubuntu2
Architecture: amd64
Config_pcmanfm_System_Lubuntu: Error: [Errno 2] No such file or directory: '/etc/xdg/pcmanfm/Lubuntu/lubuntu.conf'
CurrentDesktop: LXDE
Date: Thu May 5 10:56:31 2016
EcryptfsInUse: Yes
InstallationDate: Installed on 2016-05-02 (2 days ago)
InstallationMedia: Lubuntu 16.04 LTS "Xenial Xerus" - Release amd64 (20160420.1)
SourcePackage: pcmanfm
Udisks_dump: Error: [Errno 2] No such file or directory: 'udisks'
UpgradeStatus: No upgrade log present (probably fresh install)

Revision history for this message
Joonhee, Jun (qwertyjune) wrote :
Revision history for this message
niknah (hankin0) wrote :

I have a similar problem.

In pcmanfm when I browse to smb://xxx/yyy/zzz
The address bar will show smb://xxx/zzz

The address bar location is from _fm_path_display_name.
_fm_path_display_name will call fm_path_display_basename and then put the basename and path together.
But fm_path_display_basename will get the basename from disp_name which is set to "/" some times, which turns smb://xxx/yyy to smb://xxx

A quick fix is to disable the feature, if I change src/base/fm-path.c:1024...

else
 disp_base = fm_path_display_basename(path);

To...

else {
 disp = fm_path_to_str(path);
 return;
}

Maybe that disables something else that's useful, I don't know.

Personally, I'd rather see the full path than a pretty display path. Something that I can copy and paste to use in other apps.

Attached is a compiled libfm with the change.
Unzip it somewhere and run...
LD_LIBRARY_PATH=/tmp/WhereYouUnzippedIt pcmanfm

Revision history for this message
niknah (hankin0) wrote :
Revision history for this message
Launchpad Janitor (janitor) wrote :

Status changed to 'Confirmed' because the bug affects multiple users.

Changed in pcmanfm (Ubuntu):
status: New → Confirmed
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.