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 171212 - Doesn't emit EVENT_DELETED on readonly directory
Doesn't emit EVENT_DELETED on readonly directory
Status: RESOLVED WONTFIX
Product: gamin
Classification: Other
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: Gamin Maintainer(s)
Gamin Maintainer(s)
gnome[unmaintained]
: 149492 165938 171972 172455 300161 303233 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2005-03-22 13:08 UTC by Nickolay V. Shmyrev
Modified: 2018-07-01 08:47 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
tar.gz with patch and tests (1.78 KB, patch)
2005-03-24 20:11 UTC, Nickolay V. Shmyrev 		committed Details | Review

Description Nickolay V. Shmyrev 2005-03-22 13:08:15 UTC 
When nautilus tries to monitor readonly directories, say /root from normal user,
gamin emit's deleted event on that directory. I think it's not true, and there
should be some error handling like test for EACCESS or not in results of
g_dir_open call in gam_poll_scan_directory_internal.
Comment 1 Daniel Veillard 2005-03-24 16:22:15 UTC 
Sounds like a bug, yes. Would be good to see how the original FAM implementation
did about this case, well the latests version with the security patch in to
avoid listing directories which should not be readable.
Sounds trivial to make python based tests like python/tests/basic.py
to check various cases about directories and permissions.

Daniel
Comment 2 Nickolay V. Shmyrev 2005-03-24 17:24:54 UTC 
Fam 2.7 also behaves so. Thus I think it's better to create workaround this in
nautilus.
Comment 3 Nickolay V. Shmyrev 2005-03-24 20:10:14 UTC 
Sorry, I was wrong. Fam 2.7 really has expected behaviour and gamin is wrong in
this situation. This is really a bug.
Comment 4 Nickolay V. Shmyrev 2005-03-24 20:11:12 UTC 
Created attachment 39218 [details] [review]
tar.gz with patch and tests

This patch should fix the problem
Comment 5 Nickolay V. Shmyrev 2005-03-24 20:14:24 UTC 
*** Bug 149492 has been marked as a duplicate of this bug. ***
Comment 6 Nickolay V. Shmyrev 2005-03-24 20:16:08 UTC 
*** Bug 165938 has been marked as a duplicate of this bug. ***
Comment 7 Sebastien Bacher 2005-03-30 16:09:10 UTC 
*** Bug 171972 has been marked as a duplicate of this bug. ***
Comment 8 Sebastien Bacher 2005-04-03 17:54:32 UTC 
*** Bug 172455 has been marked as a duplicate of this bug. ***
Comment 9 Daniel Veillard 2005-04-04 11:54:17 UTC 
Oops , I nearly forgot about that issue.
I applied the patch, added the new regression tests and commited.
Unfortunately this doesn't look complete really, as this break one
of the python dnotify regression tests:

dnotify4.py
-- dnotify4.py
Error got event 2 expected 8

 so more checking and fixes may be needed

Daniel
Comment 10 Daniel Veillard 2005-04-04 13:06:44 UTC 
W.r.t. dnotify4.py regression failure, the code fails now when monitoring a
file as a directory. It report is as EXIST while it should not, the resource
is not available as a directory, I think this is wrong and need to be fixed
i.e. Deleted should be reported, not Exist in that case.

Daniel
Comment 11 Sebastien Bacher 2005-04-13 08:55:14 UTC 
*** Bug 300161 has been marked as a duplicate of this bug. ***
Comment 12 Sebastien Bacher 2005-05-06 14:19:13 UTC 
*** Bug 303233 has been marked as a duplicate of this bug. ***
Comment 13 André Klapper 2018-07-01 08:47:01 UTC 
gamin is not under active development anymore and has not seen code changes for many years.
Its codebase has been archived: https://gitlab.gnome.org/Archive/gamin/commits/master

Closing this report as WONTFIX as part of Bugzilla Housekeeping to reflect reality. Please feel free to reopen this ticket (or rather transfer the project to GNOME Gitlab, as GNOME Bugzilla is deprecated) if anyone takes the responsibility for active development again.