Ticket #42 (new defect)

Opened 9 years ago

Last modified 4 years ago

savannah: smbfs performance suffers badly if no username is specified in the URL

Reported by: ptsekov Owned by:
Priority: major Milestone: Future Releases
Component: mc-vfs Version:
Keywords: Cc:
Blocked By: #1 Blocking:
Branch state: no branch Votes for changeset:

Description (last modified by ossi) (diff)

Original: http://savannah.gnu.org/bugs/?16216

Submitted by:Pavel Tsekov <ptsekov>Submitted on:Wed 29 Mar 2006 03:07:47 PM UTC
Category:VFSSeverity:3 - Normal
Status:NonePrivacy:Public
Assigned to:NoneOpen/Closed:Open
Release:current (CVS or snapshot)Operating System:All

Original submission:

1) From the Left or Right menu select SMB Link

2) Type the hostname of a remote samba server

3) \\Password for \\xxx.xxx.xxx.xxx\IPC$ dialog appears. In the 
Username field type GUEST

4) Navigate to a share and press Enter. Notice how long it takes to 
load the listing

5) Go to a .tar.gz file and hit Enter -> MC will crash

This problem is caused by the fact that smbfs uses a pool of active 
connections which are uniquely identified by their hostname, 
username and sharename. When no user is specified in the url, but 
in the "Password for" dialog and this name is different from the 
current login name, smbfs thinks that we are trying to connect to 
the remote machine with the current login name. This confuses smbfs 
and it doesn't notice that there is a valid connection which can be 
reused - the result is that smbfs opens a new connection each time 
it needs to issue a command to the samba server.
	Pavel Tsekov <ptsekov>
Project Administrator

Comment 1 by Pavel Tsekov <ptsekov> at Thu 30 Mar 2006 09:10:24 AM UTC:

I am attaching a patch.

Attachments

mc-smbfs-dont-trash-connections.patch (5.1 KB) - added by slavazanko 9 years ago.

Change History

Changed 9 years ago by slavazanko

comment:1 Changed 9 years ago by slavazanko

  • Description modified (diff)

comment:2 Changed 9 years ago by styx

  • Milestone set to future releases

comment:3 Changed 7 years ago by andrew_b

  • severity set to no branch
  • Blocked By 1 added

comment:4 Changed 4 years ago by ossi

  • Description modified (diff)
  • Branch state set to no branch
  • Reporter changed from slavazanko to ptsekov
Note: See TracTickets for help on using tickets.