Menu

#5817 Don't add . to PATH in Make

Fixed
Maintainability
2020-03-06
2020-03-04
No

Don't add . to PATH in Make

That was apparently done by accident and may draw unexpected
executable files into the action.

http://codereview.appspot.com/563650043

Discussion

  • Anonymous

    Anonymous - 2020-03-04
    • Description has changed:

    Diff:

    
    
    • Needs: -->
    • Patch: new --> review
     
  • Anonymous

    Anonymous - 2020-03-04

    Passes make, make check and a full make doc.

     
  • David Kastrup

    David Kastrup - 2020-03-04
    • Description has changed:

    Diff:

    --- old
    +++ new
    @@ -1,5 +1,6 @@
     Don't add . to PATH in Make
    
    -That's a really big security no-no.
    
    +    That was apparently done by accident and may draw unexpected
    +    executable files into the action.
    
     http://codereview.appspot.com/563650043
    
     
  • Anonymous

    Anonymous - 2020-03-05
    • Patch: review --> countdown
     
  • Anonymous

    Anonymous - 2020-03-05

    Patch on countdown for March 7th

     
  • David Kastrup

    David Kastrup - 2020-03-05

    I am fast-tracking this one. Reasons: it is obviously™ right. And another issue of mine ready to get into review critically depends on it. Which is how I discovered it.

     
  • David Kastrup

    David Kastrup - 2020-03-06
    • labels: --> Fixed_2_21_0
    • status: Started --> Fixed
    • Patch: countdown -->
    • Type: Enhancement --> Maintainability
     
  • David Kastrup

    David Kastrup - 2020-03-06

    Pushed to staging as
    commit 825dd87d0b1b58e56d7c66ef1fc1dd672d913c84
    Author: David Kastrup dak@gnu.org
    Date: Wed Mar 4 03:10:37 2020 +0100

    Issue 5817: Don't add . to PATH in Make
    
    That was apparently done by accident and may draw unexpected
    executable files into the action.
    
     
MongoDB Logo MongoDB