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 165530 - hang after removing directory containing files
hang after removing directory containing files
Status: RESOLVED FIXED
Product: nautilus
Classification: Core
Component: general
2.9.x
Other All
: Normal major
: ---
Assigned To: Nautilus Maintainers
Nautilus Maintainers
Depends on:
Blocks:
 
 
Reported: 2005-01-28 18:30 UTC by William Jon McCann
Modified: 2005-05-18 11:37 UTC
See Also:
GNOME target: ---
GNOME version: 2.9/2.10



Description William Jon McCann 2005-01-28 18:30:52 UTC
Please describe the problem:
Open burn:///
Create directory "foo" in burn:///
Open directory "foo"
Create file "bar" in "foo"
Close directory "foo"
Remove directory "foo" using nautilus

The directory is removed from burn:/// but the CPU goes to 100% and stays there
while the burn:/// location is open.

Steps to reproduce:


Actual results:


Expected results:


Does this happen every time?


Other information:
Comment 1 William Jon McCann 2005-01-28 19:32:05 UTC
Here are the method calls made on mapping_method immediately after using the Del
key to remove the directory burn:///foo:

do_get_file_info: burn:///foo
do_get_file_info: file /foo result 25
do_get_file_info: burn:///foo
do_get_file_info: file /foo result 25
do_open_directory: burn:///foo
do_read_directory: 0x8f0ac28
do_open_directory: burn:///foo/bar
do_read_directory: 0x8f23798
do_close_directory: 0x8f23798
do_read_directory: 0x8f0ac28
do_close_directory: 0x8f0ac28
do_get_file_info: burn:///foo
do_get_file_info: file /foo result 25
do_open_directory: burn:///foo
do_read_directory: 0x8f0ac28
do_open_directory: burn:///foo/bar
do_read_directory: 0x8f23798
do_close_directory: 0x8f23798
do_remove_directory: burn:///foo/bar
do_open_directory: burn:///foo
do_read_directory: 0x8f0ac28
do_close_directory: 0x8f0ac28
do_remove_directory: burn:///foo
do_read_directory: 0x8f0ac28
do_close_directory: 0x8f0ac28
do_get_file_info: burn:///foo
do_get_file_info: file /foo result 1
do_get_file_info: burn:///foo
do_get_file_info: file /foo result 1
[Continues to loop]
Comment 2 William Jon McCann 2005-01-28 22:05:18 UTC
This seems to stop the looping:

Index: libnautilus-private/nautilus-directory-async.c
===================================================================
RCS file: /cvs/gnome/nautilus/libnautilus-private/nautilus-directory-async.c,v
retrieving revision 1.212
diff -p -u -r1.212 nautilus-directory-async.c
--- libnautilus-private/nautilus-directory-async.c      22 Nov 2004 15:24:35
-0000      1.212
+++ libnautilus-private/nautilus-directory-async.c      28 Jan 2005 21:47:04 -0000
@@ -2809,7 +2809,8 @@ get_info_callback (GnomeVFSAsyncHandle *
                nautilus_file_update_info (get_info_file, result->file_info,
has_slow_mime_type);
        }

-       nautilus_file_changed (get_info_file);
+       if (!get_info_file->details->is_gone)
+               nautilus_file_changed (get_info_file);
        nautilus_file_unref (get_info_file);

        async_job_end (directory, "file info");
Comment 3 William Jon McCann 2005-01-28 22:07:57 UTC
Not sure if this is a proper fix or just a workaround.  Moving to nautilus for
comment.
Comment 4 Sebastien Bacher 2005-01-30 16:14:19 UTC
could you mail the patch to the nautilus-list to get it reviewed ?
Comment 5 Sebastien Bacher 2005-05-08 18:38:28 UTC
any news on this bug? have you mailed the list?
Comment 6 Alexander Larsson 2005-05-16 15:11:59 UTC
Should be fixed with this:

2005-05-16  Alexander Larsson  <alexl@redhat.com>

	* libnautilus-private/nautilus-bookmark.c:
	(bookmark_file_changed_callback):
	Don't loop when a bookmarked file is removed
	
	* src/file-manager/fm-directory-view.c: (process_new_files):
	Don't call get_file_info for files no longer in the directory
	(was caused by the ready_to_load call).

	This fixes #165530 and likely #170761
Comment 7 dinoop.thomas 2005-05-18 11:34:11 UTC
I could still see this problem with the recent change.
Comment 8 dinoop.thomas 2005-05-18 11:37:23 UTC
Oops, Sorry i wanted to update for the bug #170761.

Please ignore the comment #7. 

Thanks.