bugfindutils - Bugs: bug #54171, find -depth skips unreadable...

 
 

bug #54171: find -depth skips unreadable directories

Submitter:  Tavian Barnes <tavianator>
Submitted:  Sat 23 Jun 2018 10:34:28 PM UTC
 
Category:  find Severity:  4 - Important
Item Group:  Wrong result Status:  Fixed
Privacy:  Public Assigned to:  berny
Originator Name:  Open/Closed:  Closed
Release:  4.6.0 Fixed Release:  4.7.0
* Mandatory Fields

Add a New Comment (Rich Markup)
   

Comment Type & Canned Response:
       

 

Sat 07 Jul 2018 11:00:25 PM UTC, comment #4: 

great, pushed with the fix at

  https://git.sv.gnu.org/cgit/findutils.git/commit/?id=df4610d08d

Thanks & have a nice day,
Berny

Bernhard Voelker <berny>
Project Administrator
Sat 30 Jun 2018 11:31:51 AM UTC, comment #3: 

Hello,

find attached a patch for extending the testsuite to cover this bug.

(file #44465)

Andreas Metzler <ametzler>
Thu 28 Jun 2018 11:10:21 AM UTC, comment #2: 

The attached seems to fix it, however, I didn't add any tests
etc. yet (and won't have time this weekend to do it).
Comments?

(file #44452)

Bernhard Voelker <berny>
Project Administrator
Sun 24 Jun 2018 09:57:35 AM UTC, comment #1: 

Just ran through all the binaries I had lying around: The problem is not present in oldfind, but was already present in 4.5.15.

Andreas Metzler <ametzler>
Sat 23 Jun 2018 10:34:28 PM UTC, original submission:  

If find doesn't have read permission for a directory, it skips it in -depth mode, but not in normal mode:

$ mkdir foo
$ touch foo/bar
$ chmod -r foo
$ find foo -depth
find: ‘foo’: Permission denied
$ find foo
foo
find: ‘foo’: Permission denied

Tavian Barnes <tavianator>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

Attached Files
file #44465:  0001-tests-test-find-depth-with-unreadable-directory.patch added by ametzler (2KiB - text/x-diff - Testcase for the bug)
file #44452:  54171-ftsfind-depth.diff added by berny (893B - text/x-patch - patch, not tested very much yet)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by berny (Updated the item)
  • -email is unavailable- added by jay (Updated the item)
  • -email is unavailable- added by tavianator (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only logged-in users can vote.

     

     

     

     

    Follow 7 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2019-08-29 jay Open/ClosedOpen Closed
    2019-08-29 berny Fixed ReleaseNone 4.7.0
    2018-07-07 berny StatusNone Fixed
        Assigned toNone berny
    2018-06-30 ametzler Attached File- Added 0001-tests-test-find-depth-with-unreadable-directory.patch, #44465
    2018-06-28 berny Attached File- Added 54171-ftsfind-depth.diff, #44452
    2018-06-27 jay Severity3 - Normal 4 - Important

    Back to the top


    Powered by Savane 3.9