[bugs #11175] find does unwanted permission check on pruned directory

[James Youngman]

This mail is an automated notification from the bugs tracker
 of the project: findutils.

/**************************************************************************/
[bugs #11175] Latest Modifications:

Changes by: 
                James Youngman <address@hidden>
'Date: 
                Sun 12/05/04 at 17:20 (GMT)

            What     | Removed                   | Added
---------------------------------------------------------------------------
              Status | Open                      | Closed
       Fixed Release | None                      | 4.2.9


------------------ Additional Follow-up Comments ----------------------------
You can download a release of findutils in which this problem is
fixed from ftp://alpha.gnu.org/gnu/findutils.

The releases on alpha.gnu.org are for testing purposes, so please
take the time to download the release and verify that your
problem has been solved.  Once the release has been sufficiently
tested, it can be uploaded to ftp.gnu.org for everybody to use it.







/**************************************************************************/
[bugs #11175] Full Item Snapshot:

URL: <http://savannah.gnu.org/bugs/?func=detailitem&item_id=11175>
Project: findutils
Submitted by: Geoff Clare
On: Tue 11/30/04 at 11:56

Category:  find
Severity:  5 - Average
Item Group:  None
Resolution:  Fixed
Privacy:  Public
Assigned to:  jay
Originator Name:  
Originator Email:  
Status:  Closed
Release:  4.2.8
Fixed Release:  4.2.9


Summary:  find does unwanted permission check on pruned directory

Original Submission:  Find reports an error in the following situation:

$ mkdir noxdir
$ chmod a-x noxdir
$ find * -prune -type d -print
find: noxdir: Permission denied
$ echo $?
1

Since the directory will be pruned, find should not be checking its 
permissions.  It should write "noxdir" to stdout (and its exit status should be 
0).

Follow-up Comments
------------------


-------------------------------------------------------
Date: Sun 12/05/04 at 17:20         By: James Youngman <jay>
You can download a release of findutils in which this problem is
fixed from ftp://alpha.gnu.org/gnu/findutils.

The releases on alpha.gnu.org are for testing purposes, so please
take the time to download the release and verify that your
problem has been solved.  Once the release has been sufficiently
tested, it can be uploaded to ftp.gnu.org for everybody to use it.


-------------------------------------------------------
Date: Sun 12/05/04 at 16:53         By: James Youngman <jay>
The duplicate error message is now fixed in the CVS code.  As you pointed out, 
it names the wrong directory anyway.   Oops.   Thanks for catching that.

-------------------------------------------------------
Date: Sun 12/05/04 at 16:19         By: Geoff Clare <geoffclare>
Okay, I have played a bit with the CVS version.  The only problem I found was 
some strangeness in error messages.  For example:

$ mkdir -p /var/tmp/findtest/noxdir
$ chmod a-x /var/tmp/findtest/noxdir
$ gfind-cvs /var/tmp/findtest
/var/tmp/findtest
/var/tmp/findtest/noxdir
gfind-cvs: /var/tmp/findtest: Permission denied
gfind-cvs: /var/tmp/findtest/noxdir: Permission denied
$ gfind-4.2.8 /var/tmp/findtest
/var/tmp/findtest
/var/tmp/findtest/noxdir
gfind-4.2.8: /var/tmp/findtest/noxdir: Permission denied

There is a spurious extra message about permissions for /var/tmp/findtest with 
the CVS version.  It only happens when x permission is denied and r permission 
is allowed.  (With r permission denied only the normal message is written.)



-------------------------------------------------------
Date: Sat 12/04/04 at 13:19         By: James Youngman <jay>
I have now fixed this in CVS.   The code change is small but has a profound 
effect on the treatment of the files specified on the find command line (they 
are now treated exactly like any other directory we discover during a search, 
except for what we do when we leave a subdirectory immediately below them).   
Would you be in a position to exercise the CVS version of find a bit before I 
make a proper release with this bugfix in it?












For detailed info, follow this link:
<http://savannah.gnu.org/bugs/?func=detailitem&item_id=11175>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/