]> git.ipfire.org Git - thirdparty/suricata.git/log
thirdparty/suricata.git
9 years agoqa: add leak sanitzer suppression 1851/head
Victor Julien [Fri, 12 Feb 2016 08:48:05 +0000 (09:48 +0100)] 
qa: add leak sanitzer suppression

9 years agowirefuzz: exit with error code on more issues
Victor Julien [Thu, 11 Feb 2016 17:51:15 +0000 (18:51 +0100)] 
wirefuzz: exit with error code on more issues

9 years agowirefuzz: logdir handling improvement
Victor Julien [Thu, 11 Feb 2016 08:46:44 +0000 (09:46 +0100)] 
wirefuzz: logdir handling improvement

9 years agowirefuzz: remove obsolete stream mem check
Victor Julien [Thu, 26 Nov 2015 17:59:34 +0000 (18:59 +0100)] 
wirefuzz: remove obsolete stream mem check

9 years agowirefuzz: add -N option to count complete passes
Victor Julien [Thu, 26 Nov 2015 16:32:25 +0000 (17:32 +0100)] 
wirefuzz: add -N option to count complete passes

9 years agowirefuzz: improve logfile cleanup
Victor Julien [Thu, 26 Nov 2015 13:37:10 +0000 (14:37 +0100)] 
wirefuzz: improve logfile cleanup

9 years agowirefuzz: enforce -n option per run
Victor Julien [Thu, 26 Nov 2015 12:42:43 +0000 (13:42 +0100)] 
wirefuzz: enforce -n option per run

9 years agowirefuzz: add -S option for exclusive rule load
Victor Julien [Thu, 26 Nov 2015 12:29:05 +0000 (13:29 +0100)] 
wirefuzz: add -S option for exclusive rule load

9 years agohttp: fix multipart body tracking slowdown
Victor Julien [Fri, 12 Feb 2016 15:31:57 +0000 (16:31 +0100)] 
http: fix multipart body tracking slowdown

Optimize HTTP multipart body parsing. Big records that were not files
could slow down Suricata. The reason was that the body tracker was not
moved forward. This lead to growing body buffers, which were expensive
wrt memory and inspection.

This patch add logic to move the tracker forward in this case.

9 years agoasan: fix reputation code include
Victor Julien [Thu, 11 Feb 2016 17:19:34 +0000 (18:19 +0100)] 
asan: fix reputation code include

9 years agotls-sni: fix uninitialized memory use
Victor Julien [Fri, 12 Feb 2016 09:54:02 +0000 (10:54 +0100)] 
tls-sni: fix uninitialized memory use

On bad traffic the parser could allocated memory that was not
intialized. This was later used in the JSON output logging as
a valid null terminated string.

9 years agopool: fix memory leak
Victor Julien [Fri, 12 Feb 2016 09:48:26 +0000 (10:48 +0100)] 
pool: fix memory leak

Due to pointer size mishandling, the pool code could consider a
block of memory inside the 'preallocated' block. It would then not
free the block.

9 years agostream: improve StreamTcpSegmentForEach for IPS 1846/head
Victor Julien [Wed, 3 Feb 2016 08:51:33 +0000 (09:51 +0100)] 
stream: improve StreamTcpSegmentForEach for IPS

StreamTcpSegmentForEach would only return ACK'd segments. This lead
to missing stream data in alerts when running in IPS mode.

This patch changes the behavior for IPS. All segments are iterated
now, also the non-ACK'd ones. For IDS mode the behavior is unchanged.

9 years agoqa: add --simulate-ips option
Victor Julien [Sun, 31 Jan 2016 18:27:35 +0000 (19:27 +0100)] 
qa: add --simulate-ips option

This option forces the engine in 'IPS' mode. This is useful for testing
some IPS code paths based on pcaps.

9 years agoeve: fix stream payload logging wrong direction
Victor Julien [Sun, 31 Jan 2016 12:40:07 +0000 (13:40 +0100)] 
eve: fix stream payload logging wrong direction

In the EVE stream payload logging the IPS path logged the wrong dir.
Both IDS and IPS can take the same path as the detection engine
inspects in the same direction in both cases, so the alert is also
generated in the same direction.

Bug #1684

9 years agoAdded checking of negated "totals" and "threads" config values for stats. 1845/head
Maxtors [Mon, 1 Feb 2016 13:54:29 +0000 (14:54 +0100)] 
Added checking of negated "totals" and "threads" config values for stats.

9 years agolua: expose TLS certificate chain to lua
Mats Klepsland [Mon, 2 Nov 2015 12:21:37 +0000 (13:21 +0100)] 
lua: expose TLS certificate chain to lua

Expose TLS certificate chain to lua through TlsGetCertChain().

9 years agofile-store: add force-filestore configuration option to enable writing all
Tom DeCanio [Mon, 5 Oct 2015 22:34:55 +0000 (15:34 -0700)] 
file-store: add force-filestore configuration option to enable writing all
            extracted files to filesystem.

9 years agolua: set thread vars in DetectLuaMatch
Nicolas Thill [Fri, 5 Feb 2016 10:11:13 +0000 (11:11 +0100)] 
lua: set thread vars in DetectLuaMatch

Fix internal error when calling SCThreadInfo from Lua detection modules.

Signed-off-by: Nicolas Thill <ntl@p1sec.com>
9 years agofilestore-call: forcing a call to FileStore instead of manually updating
Maurizio Abba [Mon, 16 Nov 2015 12:21:27 +0000 (12:21 +0000)] 
filestore-call: forcing a call to FileStore instead of manually updating
the relative flag in order to have a single point where we actually
touch the File structure

9 years agofix nfq_get_timestamp
notnyt [Sat, 6 Feb 2016 00:11:03 +0000 (19:11 -0500)] 
fix nfq_get_timestamp

Handle case when nfq_get_timestamp returns 0 for success, but timestamp is empty.

9 years agoFix typo of trailing ] in configure --help
bladeswords [Wed, 3 Feb 2016 11:09:34 +0000 (22:09 +1100)] 
Fix typo of trailing ] in configure --help

It is the small things that count.  This is an example of the fix

Before
--disable-threading-tls Disable TLS (thread local storage)]

After
--disable-threading-tls Disable TLS (thread local storage)

9 years agoFix the comment and explanation for random-chunk-size
Andreas Herz [Tue, 17 Nov 2015 11:25:29 +0000 (12:25 +0100)] 
Fix the comment and explanation for random-chunk-size

9 years agojson: fix missing includes in disable unix socket case 1842/head
Andreas Herz [Sun, 7 Feb 2016 22:02:14 +0000 (23:02 +0100)] 
json: fix missing includes in disable unix socket case

9 years agodrmemory: suppress magic leak
Victor Julien [Mon, 8 Feb 2016 10:50:20 +0000 (11:50 +0100)] 
drmemory: suppress magic leak

9 years agofile-magic: improve libmagic handling on *nix systems
Andreas Herz [Fri, 29 Jan 2016 21:02:22 +0000 (22:02 +0100)] 
file-magic: improve libmagic handling on *nix systems

9 years agoIn configuration test mode, check signatures if 'delayed-detect' is enabled
Alexander Gozman [Thu, 4 Feb 2016 12:33:23 +0000 (15:33 +0300)] 
In configuration test mode, check signatures if 'delayed-detect' is enabled

When 'detection-engine.delayed-detect' option was set to 'yes',
suricata didn't check signatures validity in configuration test mode.

9 years agoremove unnecessary braces
Andreas Herz [Sun, 31 Jan 2016 21:34:56 +0000 (22:34 +0100)] 
remove unnecessary braces

9 years agoconfigure: add --disable-python option
Andreas Herz [Sun, 31 Jan 2016 22:15:20 +0000 (23:15 +0100)] 
configure: add --disable-python option

9 years agoasn1: fix memory leak
Jason Ish [Wed, 3 Feb 2016 20:28:50 +0000 (14:28 -0600)] 
asn1: fix memory leak

As reported in issue #1395, fix 2 memory leaks when destroying
asn.1 decode contexts.

9 years agoFix two more potential issues like bug 1550
Victor Julien [Tue, 22 Sep 2015 13:20:25 +0000 (15:20 +0200)] 
Fix two more potential issues like bug 1550

9 years agoSegfault fix
Alessandro Guido [Wed, 16 Sep 2015 13:14:11 +0000 (15:14 +0200)] 
Segfault fix

9 years agococcinelle: fix typo for strchrnul 1821/head
Andreas Herz [Tue, 19 Jan 2016 20:38:37 +0000 (21:38 +0100)] 
coccinelle: fix typo for strchrnul

9 years agoconf: null guard in ConfNodeLookupChild
Jason Ish [Thu, 21 Jan 2016 04:41:22 +0000 (22:41 -0600)] 
conf: null guard in ConfNodeLookupChild

Add NULL guarding to the lookup so callers can process nodes
in a loop with less error checking.

Addresses issue #1660.

9 years agounix manager: log client's version with debug level
Alexander Gozman [Wed, 23 Dec 2015 08:49:11 +0000 (11:49 +0300)] 
unix manager: log client's version with debug level

As (dis)connects are already logged as a debug events, this one
should do the same.

9 years agounix-manager: fix ressource leak when init fail
Eric Leblond [Mon, 21 Dec 2015 14:13:09 +0000 (15:13 +0100)] 
unix-manager: fix ressource leak when init fail

9 years agoUpdate Changelog for 3.0 suricata-3.0
Victor Julien [Fri, 22 Jan 2016 15:56:48 +0000 (16:56 +0100)] 
Update Changelog for 3.0

9 years agosmtp: harden mime parsing 1812/head
Victor Julien [Mon, 11 Jan 2016 21:46:35 +0000 (16:46 -0500)] 
smtp: harden mime parsing

9 years agoUpdate changelog for 3.0RC3 suricata-3.0RC3
Victor Julien [Mon, 21 Dec 2015 13:11:44 +0000 (14:11 +0100)] 
Update changelog for 3.0RC3

9 years agoredis: fix compiler warning 1799/head
Victor Julien [Sat, 19 Dec 2015 15:27:35 +0000 (16:27 +0100)] 
redis: fix compiler warning

9 years agoprelude: clean up memory on setup failure
Victor Julien [Fri, 18 Dec 2015 10:07:36 +0000 (11:07 +0100)] 
prelude: clean up memory on setup failure

9 years agopcap: fix setup failure memleak on libpcap < 1.0
Victor Julien [Fri, 18 Dec 2015 10:03:40 +0000 (11:03 +0100)] 
pcap: fix setup failure memleak on libpcap < 1.0

9 years agoprelude: style fixup
Victor Julien [Thu, 17 Dec 2015 15:41:05 +0000 (16:41 +0100)] 
prelude: style fixup

9 years agoprelude: fix thread safeness
Eric Leblond [Mon, 7 Dec 2015 11:08:16 +0000 (12:08 +0100)] 
prelude: fix thread safeness

Prelude analyzer is not thread safe so we need to have one
analyzer per thread.

9 years agosuricata: can't use -l and unix socket runmode
Eric Leblond [Mon, 14 Dec 2015 14:18:07 +0000 (15:18 +0100)] 
suricata: can't use -l and unix socket runmode

It is not possible to use simultaneously -l and unix socket
runmode because setting the log directory make it final so
not modifable by other call.

It is a implementation limitation but it does not make sense
to set logging directory to have it overwritten by the first
directory specified 'by pcap-file'. So it seems correct to
only trigger an error if this both options are used at the same
time.

9 years agounix-manager: display errors like errors
Eric Leblond [Mon, 14 Dec 2015 14:04:47 +0000 (15:04 +0100)] 
unix-manager: display errors like errors

Not being able to setup the output directory or file is an error so
it should be printed as such.

9 years agounix-manager: fix race condition
Eric Leblond [Mon, 14 Dec 2015 13:02:20 +0000 (14:02 +0100)] 
unix-manager: fix race condition

Under high load it is possible that the thread is not yet started
and that we register a command at the same time. As a consequence,
the commands list is not yet initialized and we have a segfault.

This patch moves the initialization in the ThreadInit function to
be sure the commands list is available when needed.

9 years agoicmpv4: remove unused header fields
Victor Julien [Wed, 16 Dec 2015 10:02:15 +0000 (11:02 +0100)] 
icmpv4: remove unused header fields

9 years agoicmpv4: remove unused declarations
Victor Julien [Wed, 16 Dec 2015 09:49:50 +0000 (10:49 +0100)] 
icmpv4: remove unused declarations

9 years agoicmpv4: harden embedded packet handling
Victor Julien [Wed, 16 Dec 2015 09:50:50 +0000 (10:50 +0100)] 
icmpv4: harden embedded packet handling

9 years agoicmpv4: improve dest unreachable logic
Victor Julien [Wed, 16 Dec 2015 09:45:05 +0000 (10:45 +0100)] 
icmpv4: improve dest unreachable logic

When a ICMPv4 destination unreachable packet contains an embedded packet
this packet is parsed. When it's found to be invalid, the whole ICMP
packet is tagged as invalid.

In some cases the unreachable packet would still be used.

This patch fixes this by checking the packet is invalid flag as well
in the ICMPV4_DEST_UNREACH_IS_VALID macro.

9 years agoDER decoder: don't use strlcpy on non-strings
Victor Julien [Thu, 17 Dec 2015 12:25:02 +0000 (13:25 +0100)] 
DER decoder: don't use strlcpy on non-strings

9 years agohttp: more sane body inspection/tracking defaults 1790/head
Victor Julien [Fri, 11 Dec 2015 15:05:42 +0000 (16:05 +0100)] 
http: more sane body inspection/tracking defaults

9 years agohttp: improve body pruning
Victor Julien [Thu, 10 Dec 2015 15:12:05 +0000 (16:12 +0100)] 
http: improve body pruning

In case the body wasn't inspected the body_inspected variable wouldn't
get updated leading to the body not getting pruned at all.

This patch adds support for this case.

9 years agoips/drop-log: fix crash on logging drops
Victor Julien [Thu, 10 Dec 2015 08:58:52 +0000 (09:58 +0100)] 
ips/drop-log: fix crash on logging drops

When logging drops for fragmented UDP packets, triggered by detection
in the reassembled packet, a missing check could lead to access of the
packets UDP header pointer when it was NULL.

9 years agohttp file: fix txid type 1788/head
Victor Julien [Fri, 13 Nov 2015 09:45:18 +0000 (10:45 +0100)] 
http file: fix txid type

9 years agofile: sync file and tx id types
Victor Julien [Fri, 13 Nov 2015 09:38:21 +0000 (10:38 +0100)] 
file: sync file and tx id types

9 years agosmtp: reset inspection buffers
Victor Julien [Tue, 1 Dec 2015 14:20:19 +0000 (15:20 +0100)] 
smtp: reset inspection buffers

9 years agohttp: fix body prune check
Victor Julien [Fri, 13 Nov 2015 13:06:32 +0000 (14:06 +0100)] 
http: fix body prune check

9 years agoUpdate Changelog for 3.0RC2 suricata-3.0RC2
Victor Julien [Tue, 8 Dec 2015 13:08:18 +0000 (14:08 +0100)] 
Update Changelog for 3.0RC2

9 years agodns: reject bad response data
Victor Julien [Wed, 2 Dec 2015 10:37:57 +0000 (11:37 +0100)] 
dns: reject bad response data

9 years agosmtp: improve handling of bad traffic
Victor Julien [Mon, 30 Nov 2015 14:46:15 +0000 (15:46 +0100)] 
smtp: improve handling of bad traffic

No longer fail tracking the SMTP stream when a unexpected reply is
encountered.

Do not store the unexpected reply.

9 years agostream-tcp: bail early on segments before base_seq
Victor Julien [Fri, 27 Nov 2015 16:12:52 +0000 (17:12 +0100)] 
stream-tcp: bail early on segments before base_seq

In cases where base_seq has moved beyond last_ack, bail early.

9 years agotls: suppress warnings on alloc failure 1786/head
Victor Julien [Mon, 7 Dec 2015 16:10:40 +0000 (17:10 +0100)] 
tls: suppress warnings on alloc failure

9 years agounified2: disable by default 1784/head
Victor Julien [Sun, 6 Dec 2015 09:53:42 +0000 (10:53 +0100)] 
unified2: disable by default

9 years agostats log: suppress 0 counters by default
Victor Julien [Sun, 8 Nov 2015 08:55:00 +0000 (09:55 +0100)] 
stats log: suppress 0 counters by default

9 years agoafpacket: suppress output
Victor Julien [Sun, 8 Nov 2015 08:18:17 +0000 (09:18 +0100)] 
afpacket: suppress output

9 years agoafpacket: move zero copy setup to config parsing
Victor Julien [Sun, 8 Nov 2015 08:06:03 +0000 (09:06 +0100)] 
afpacket: move zero copy setup to config parsing

This way it's run and logged per device, instead of per thread.

9 years agoafpacket: indent fixup
Victor Julien [Sun, 8 Nov 2015 08:01:52 +0000 (09:01 +0100)] 
afpacket: indent fixup

9 years agooffloading: compress printing of iface offloading
Victor Julien [Sun, 8 Nov 2015 07:58:26 +0000 (08:58 +0100)] 
offloading: compress printing of iface offloading

9 years agoafpacket: on missing cluster settings, set defaults
Victor Julien [Sun, 8 Nov 2015 07:48:24 +0000 (08:48 +0100)] 
afpacket: on missing cluster settings, set defaults

9 years agoafpacket: add null decoder, put ethernet first
Victor Julien [Sat, 7 Nov 2015 20:24:07 +0000 (21:24 +0100)] 
afpacket: add null decoder, put ethernet first

9 years agodevice: constify string args
Victor Julien [Sat, 7 Nov 2015 18:28:25 +0000 (19:28 +0100)] 
device: constify string args

9 years agomodbus: disable by default
Jason Ish [Thu, 3 Dec 2015 21:40:08 +0000 (15:40 -0600)] 
modbus: disable by default

9 years agooutput-json: add app_proto key in root
Eric Leblond [Thu, 3 Dec 2015 11:07:03 +0000 (12:07 +0100)] 
output-json: add app_proto key in root

By adding the key in the root of *flow and fileinfo  events it
will be possible to get all events for one application layer by
using a 'event_type:proto OR app_proto:proto' filter. This will
permit to the analyst to get a good view of events related to
one protocol.

This patch also fixes a regression in file logging where app_proto
was available before 94dbd303e4744a40f3761265be7c73a7a4754764 create
the regression.

9 years agoFeature 1605: more descriptive error messages when checking MTU, etc 1774/head
Alexander Gozman [Mon, 30 Nov 2015 08:21:05 +0000 (11:21 +0300)] 
Feature 1605: more descriptive error messages when checking MTU, etc

9 years agoprofiling: fix lock profile compilation 1772/head
Victor Julien [Fri, 27 Nov 2015 08:50:50 +0000 (09:50 +0100)] 
profiling: fix lock profile compilation

9 years agojson: small improvement to log message wording 1765/head
Victor Julien [Thu, 26 Nov 2015 12:03:11 +0000 (13:03 +0100)] 
json: small improvement to log message wording

9 years agooutput-json: fix regression on log prefix handling
Eric Leblond [Mon, 23 Nov 2015 12:46:18 +0000 (13:46 +0100)] 
output-json: fix regression on log prefix handling

The log prefix option was not anymore honored due to a regression
caused by some recent code.

9 years agoapp-layer-smtp: support for multiline response
Maurizio Abba [Mon, 16 Nov 2015 12:14:24 +0000 (12:14 +0000)] 
app-layer-smtp: support for multiline response
Multiline response support is provided but not enforced. This patch
allow parsing multiline response when a reply is processed

9 years agoUpdate changelog for 3.0RC1 suricata-3.0RC1
Victor Julien [Wed, 25 Nov 2015 12:03:40 +0000 (13:03 +0100)] 
Update changelog for 3.0RC1

9 years agoUpdate dev version to reflect we're doing 3.0 now
Victor Julien [Wed, 25 Nov 2015 12:02:48 +0000 (13:02 +0100)] 
Update dev version to reflect we're doing 3.0 now

9 years agorule vars: fix compiler warning
Victor Julien [Wed, 25 Nov 2015 11:17:51 +0000 (12:17 +0100)] 
rule vars: fix compiler warning

9 years agotls: fix compiler warnings 1762/head
Victor Julien [Tue, 24 Nov 2015 11:06:04 +0000 (12:06 +0100)] 
tls: fix compiler warnings

9 years agohttp: add test for plain http over connect
Victor Julien [Sat, 11 Jul 2015 09:43:34 +0000 (11:43 +0200)] 
http: add test for plain http over connect

9 years agohttp: don't run unittests twice
Victor Julien [Mon, 13 Jul 2015 09:26:41 +0000 (11:26 +0200)] 
http: don't run unittests twice

9 years agohttp: test cleanups
Victor Julien [Sat, 11 Jul 2015 20:52:23 +0000 (22:52 +0200)] 
http: test cleanups

9 years agoFix out-of-bounds memory access in DNS TXT record parser.
Aaron Campbell [Mon, 2 Nov 2015 19:19:12 +0000 (15:19 -0400)] 
Fix out-of-bounds memory access in DNS TXT record parser.

The datalen variable is declared unsigned.  If txtlen and datalen are equal,
datalen will first be reduced to 0, and then the datalen-- line will cause its
value to wrap to 65535.  This will cause the loop to continue much longer than
intended, and eventually may crash on an out-of-bounds *tdata dereference.

Signed-off-by: Aaron Campbell <aaron@monkey.org>
9 years agomulti-detect: fix and simplify config 1760/head
Victor Julien [Mon, 23 Nov 2015 16:18:57 +0000 (17:18 +0100)] 
multi-detect: fix and simplify config

instead

mappings:
  - vlan:
    vlan-id: 1
    tenant-id: 2

we'll now use:

mappings:
  - vlan-id: 1
    tenant-id: 2

For YAML it pretty much means the same thing.

Ticket: 1517

9 years agomulti-detect: improve error handling
Victor Julien [Mon, 23 Nov 2015 13:58:34 +0000 (14:58 +0100)] 
multi-detect: improve error handling

9 years agomulti-detect: handle missing mappings
Victor Julien [Mon, 23 Nov 2015 13:05:21 +0000 (14:05 +0100)] 
multi-detect: handle missing mappings

Notify/warn user about missing mappings depending on other settings
like unix socket and init errors fatal.

9 years agomulti-detect: consider vlan tracking
Victor Julien [Mon, 23 Nov 2015 12:38:59 +0000 (13:38 +0100)] 
multi-detect: consider vlan tracking

Refuse to use vlan selector if vlan tracking is disabled.

9 years agomulti-detect: validate vlan_id
Victor Julien [Mon, 23 Nov 2015 12:30:03 +0000 (13:30 +0100)] 
multi-detect: validate vlan_id

9 years agomulti-detect: use default tenant
Victor Julien [Fri, 20 Nov 2015 14:52:50 +0000 (15:52 +0100)] 
multi-detect: use default tenant

The default detect engine can be used as 'default tenant'.

9 years agomulti-detect: clean up output
Victor Julien [Fri, 20 Nov 2015 11:05:02 +0000 (12:05 +0100)] 
multi-detect: clean up output

9 years agobase64: code style fixups
Victor Julien [Thu, 19 Nov 2015 12:37:23 +0000 (13:37 +0100)] 
base64: code style fixups

9 years agobase64_decode, base64_data: decode and match base64 1758/head
Jason Ish [Wed, 30 Sep 2015 15:49:47 +0000 (09:49 -0600)] 
base64_decode, base64_data: decode and match base64

9 years agoutil-base64: strict mode - all characters must be valid
Jason Ish [Wed, 14 Oct 2015 19:37:45 +0000 (13:37 -0600)] 
util-base64: strict mode - all characters must be valid

Introduce a strict mode to base64 decode. If strict,
the function will fail when invalid input data is seen.
If not strict, what has been decoded will be returned.

This is in support of adding a Snort compatible base64_decode
rule option that uses whatever data can be decoded as a length
of data to decode is optional.

9 years agolua: added function TlsGetSNI()
Mats Klepsland [Tue, 13 Oct 2015 13:59:22 +0000 (15:59 +0200)] 
lua: added function TlsGetSNI()

Added function to get server name from TLS SNI extension.

9 years agolog-tls: added SNI field to extended output
Mats Klepsland [Tue, 13 Oct 2015 13:35:33 +0000 (15:35 +0200)] 
log-tls: added SNI field to extended output

Added SNI field to extended tls log output.