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

Patch9/10 #1166

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Patch9/10 #1166

wants to merge 7 commits into from

Conversation

hcw2016
Copy link

@hcw2016 hcw2016 commented Jun 30, 2017

Patches were added to ensure the backing up of an agent's information before force deleting. Also, an alert is sent if an agent with a duplicated IP is sent to authd. Additionally, changes were made to overload the mkdir functions so they would work with windows agents. This allows the specification of file permissions to ensure security.


This change is Reviewable

cwoods added 7 commits June 28, 2017 14:35
…ion of agent with a duplicated IP, and enables the option for re-using agent ID's in authd.
…limit to force the deletion of an agent with a duplicated IP, and the option for the re-using of an agent ID in authd
…for duplicated IPs, and enabled option for re-using an agent ID
…st_mtime and removed the .tv_sec, since mtime returns the time of last modification in seconds. Should resolve previous build errors in Travis.
… alert if authd tries to add an agent with a duplicated IP. Additionally, changes were made to overload functions like mkdir to specify file permissions to ensure security.
@ddpbsd
Copy link
Member

ddpbsd commented Jul 8, 2017

Thanks for the pull request! Can you merge MASTER into your branch? It should grab the latest changes you made to patch 8.
I think this should do it:
git pull https://github.com/ossec/ossec-hids.git master

@hcw2016
Copy link
Author

hcw2016 commented Jul 10, 2017

Sure thing! If you need anything else, please let me know!

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