Skip to content
This repository has been archived by the owner on Apr 17, 2023. It is now read-only.

Fix to issue #140 #141

Merged
merged 1 commit into from
Mar 2, 2020
Merged

Fix to issue #140 #141

merged 1 commit into from
Mar 2, 2020

Conversation

rx294
Copy link
Contributor

@rx294 rx294 commented Mar 2, 2020

V-72027 contains incorrect test code

Reverting to pervious correct version

ignore_shells = non_interactive_shells.join('|')
findings = Set[]
users.where{ !shell.match(ignore_shells) && (uid >= 1000 || uid == 0)}.entries.each do |user_info|
next if exempt_home_users.include?("#{user_info.username}")
findings = findings + command("find #{user_info.home} -xdev ! -name '.*' -perm /027 ! -type l").stdout.split("\n")
end
describe "Home directories with excessive permissions" do
subject { findings.to_a }
it { should be_empty }
end

@rx294 rx294 requested a review from aaronlippold March 2, 2020 16:40
@aaronlippold aaronlippold merged commit 963595d into master Mar 2, 2020
@aaronlippold aaronlippold deleted the fix_140 branch March 2, 2020 18:24
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants