After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 144906 - Nautilus crashes when host name is not given for ftp method in Solaris
Nautilus crashes when host name is not given for ftp method in Solaris
Status: RESOLVED FIXED
Product: nautilus
Classification: Core
Component: File and Folder Operations
2.5.x
Other Solaris
: Normal normal
: ---
Assigned To: gnome-vfs maintainers
gnome-vfs maintainers
Depends on:
Blocks:
 
 
Reported: 2004-06-24 10:50 UTC by Narayana Pattipati
Modified: 2004-12-22 21:47 UTC
See Also:
GNOME target: ---
GNOME version: 2.5/2.6


Attachments
proposed patch to fix the crash. (1.66 KB, patch)
2004-06-25 15:39 UTC, Narayana Pattipati
none Details | Review

Description Narayana Pattipati 2004-06-24 10:50:51 UTC
Nautilus is crashing in when Host name is not given in the URL of the ftp method.

Steps to simulate:

a) Launch nautilus and open a directory in browser mode.
b) type "ftp://naren:naren123" in the location bar and enter.

Nautilus crashes. 

The crash does not happen in Linux. But in Linux, a dialog is given which says
"Nautilus cannot display "ftp:". Please select another viewer and try again.".
Also a commandline warning is given which says:

(nautilus:9277): gnome-vfs-modules-WARNING **: gnome_vfs_inet_connection_create
("(null)", 0) = "Invalid parameters"

I feel error dialog should say something like "no host specified..." which is
the case with "sftp".
Comment 1 Narayana Pattipati 2004-06-25 15:37:28 UTC
The changes made by Alex on 24th June returns GNOME_VFS_ERROR_INVALID_HOST. So,
the error message is taken care. 

Once gnome-vfs returns this code, nautilus has to convert this error
code into human readeable format and show a dialog. At that time,
nautilus just assumes that host_name is allocated and tries to use that
in the error message, leading to another crash in nautilus code in Solaris. 

I will attach a patch soon.
Comment 2 Narayana Pattipati 2004-06-25 15:39:40 UTC
Created attachment 29006 [details] [review]
proposed patch to fix the crash.
Comment 3 Christophe Fergeau 2004-06-28 17:45:31 UTC
You should send a mail to the nautilus mailing list
(http://mail.gnome.org/mailman/listinfo/nautilus-list) to get your patch reviewed.
Comment 4 Alexander Larsson 2004-07-02 08:15:28 UTC
host_name needs to be const char * or the build fails. (Didn't it use -Werror
when you built?)

I commited the patch with that fix.
Comment 5 Narayana Pattipati 2004-07-02 13:06:40 UTC
Alex,
Yes, I had -Werror in the flags when I built it. Thanks for that.