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

 
 

bug #54171: find -depth skips unreadable directories

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

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

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 AdministratorIn charge of this item.
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 AdministratorIn charge of this item.
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)
  •  

    Do you think this task is very important?
    If so, you can add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

     

     

    Follow 5 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    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.3