Re: [PATCH] Re: SMB module authentication rewrite patch

[Narayana Pattipati <narayana pattipati wipro com>]

Since the existing bug http://bugzilla.gnome.org/show_bug.cgi?id=132933  is still open, I have uploaded the patch there, so that it will not get lost in mails.

Please have a look at it.

Regards,
Narayana

On Wed, 2005-01-12 at 19:34, Narayana Pattipati wrote:

On Wed, 2005-01-12 at 03:30, Nielsen wrote:

Narayana Pattipati wrote:
> b) Entering smb://domain;username machine throws a password dialog but
> it never authenticated user. Even after giving correct password, the
> authentication dialog keeps coming.  This is use case while adding a 
> network place from network: location. User can either enter username or
> go without user name. If user enters a username there, url becomes as
> mentioned above, and user will encounter the problem.

True, that's odd. I'll look into it.

> I will let you know if I see any other issues with the patch.

I had a look at the code and figured out the reason for this.

The problem will be there when smb uri  is in the following forms:
- smb://username machine
- smb://domain;username machine
- smb://domain;@machine

When url is like this, while processing the uri (initial_authentication()) and while prompting for authentication (prompt_authentication()), assumptions have been made about the availability of domain name.  Same is true while caching also.

Also, after invoking the authentication for say password only (username, domain are available through url), the return values from gnome_vfs_module_callback_invoke() are filled into username and domain also, even though user is not prompted for them. User is just prompted for password. So, just fill in password.

The attached patch takes care of these issues and makes smb browsing work when url happens to be in the above forms.

Please have a look at the patch.

Regards,
Narayana

---

_______________________________________________
gnome-vfs-list mailing list
gnome-vfs-list gnome org
http://mail.gnome.org/mailman/listinfo/gnome-vfs-list