Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Keep the code to delete previously-named malware rules file #3619

Merged
merged 0 commits into from
Dec 1, 2022

Conversation

mhuth
Copy link
Contributor

@mhuth mhuth commented Nov 30, 2022

  • just in case the previously-named rules file isn't deleted for some reason

Signed-off-by: Mark Huth [email protected]

All Pull Requests:

Check all that apply:

  • Have you followed the guidelines in our Contributing document, including the instructions about commit messages?
  • Is this PR to correct an issue?
  • Is this PR an enhancement?

Just some small changes to some recent malware-detection commits I made. Main one was making sure the old downloaded rules file is still deleted. I've renamed the file, but its possible (although rare) that the old file could be maintained on the customer's system and isn't deleted as we've changed the file name now that we are deleting. This commit makes sure the old file name is still deleted as well.

@mhuth mhuth requested a review from xiangce November 30, 2022 22:26
Copy link
Contributor

@xiangce xiangce left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good to me

@xiangce xiangce merged commit b6e75ab into master Dec 1, 2022
@xiangce xiangce deleted the malware_misc_changes branch December 1, 2022 01:33
xiangce pushed a commit that referenced this pull request Dec 1, 2022
- just in case the previously-named rules file isn't deleted for some reason

Signed-off-by: Mark Huth <[email protected]>
(cherry picked from commit b6e75ab)
xiangce pushed a commit that referenced this pull request Sep 6, 2024
- just in case the previously-named rules file isn't deleted for some reason

Signed-off-by: Mark Huth <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants