Skip to content

Commit

Permalink
Add IP validation to Security module
Browse files Browse the repository at this point in the history
For event 4778 (A session was reconnected to a Window Station) the `winlog.event_data.ClientAddress`
could be "LOCAL" which is obviosuly not a valid IP so we don't want to copy it into `source.ip` in that case.

Fixes elastic#19627
  • Loading branch information
andrewkroh committed Sep 24, 2020
1 parent 1f7f9c5 commit 7c0a160
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
1 change: 1 addition & 0 deletions CHANGELOG.next.asciidoc
Original file line number Diff line number Diff line change
Expand Up @@ -363,6 +363,7 @@ https://github.com/elastic/beats/compare/v7.0.0-alpha2...master[Check the HEAD d

- Fix invalid IP addresses in DNS query results from Sysmon data. {issue}18432[18432] {pull}18436[18436]
- Fields from Winlogbeat modules were not being included in index templates and patterns. {pull}18983[18983]
- Add source.ip validation for event ID 4778 in the Security module. {issue}19627[19627]

*Functionbeat*

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -1479,11 +1479,12 @@ var security = (function () {
fields: [
{from: "winlog.event_data.AccountName", to: "user.name"},
{from: "winlog.event_data.AccountDomain", to: "user.domain"},
{from: "winlog.event_data.ClientAddress", to: "source.ip"},
{from: "winlog.event_data.ClientAddress", to: "source.ip", type: "ip"},
{from: "winlog.event_data.ClientName", to: "source.domain"},
{from: "winlog.event_data.LogonID", to: "winlog.logon.id"},
],
ignore_missing: true,
fail_on_error: false,
})
.Add(function(evt) {
var user = evt.Get("winlog.event_data.AccountName");
Expand Down

0 comments on commit 7c0a160

Please sign in to comment.