Project

General

Profile

Actions
Copy link

Task #7461

closed
PA PA

suricata-verify: pass all tests

Task #7461: suricata-verify: pass all tests

Added by Philippe Antoine over 1 year ago. Updated 9 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Philippe Antoine
Target version:
8.0.0-rc1
Effort:
Difficulty:
Label:

Related issues 5 (0 open — 5 closed)

Related to Suricata - Optimization #7416: suricata-verify: remove requires on file namesClosedOISF DevActions
Related to Suricata - Security #7450: tracking: signature can allocate arbitrary amount of memoryClosedVictor JulienActions
Blocked by Suricata - Bug #7447: NULL dereference in ThreadLogFileHashFreeFunc in bug-5198 SV testClosedJeff LucovskyActions
Blocked by Suricata - Bug #7449: app-layer metadata does not get logged for stream rules and unidirectional protocolsClosedPhilippe AntoineActions
Blocked by Suricata - Bug #7462: suricata-verify: pass when compiled with FUZZING_UNSAFERejectedPhilippe AntoineActions

PA Updated by Philippe Antoine over 1 year ago Actions
Copy link
 #1

  • Blocked by Bug #7447: NULL dereference in ThreadLogFileHashFreeFunc in bug-5198 SV test added

PA Updated by Philippe Antoine over 1 year ago Actions
Copy link
 #2

PA Updated by Philippe Antoine over 1 year ago Actions
Copy link
 #3

  • Blocks Security #7450: tracking: signature can allocate arbitrary amount of memory added

PA Updated by Philippe Antoine over 1 year ago Actions
Copy link
 #4

  • Blocked by Bug #7449: app-layer metadata does not get logged for stream rules and unidirectional protocols added

PA Updated by Philippe Antoine over 1 year ago Actions
Copy link
 #5

  • Blocks deleted (Security #7450: tracking: signature can allocate arbitrary amount of memory)

PA Updated by Philippe Antoine over 1 year ago Actions
Copy link
 #6

PA Updated by Philippe Antoine over 1 year ago Actions
Copy link
 #7

PA Updated by Philippe Antoine over 1 year ago Actions
Copy link
 #8

  • Related to Security #7450: tracking: signature can allocate arbitrary amount of memory added

PA Updated by Philippe Antoine over 1 year ago Actions
Copy link
 #9

Also
- DHCP test to enable
- Remove obsolete tests unified2 and http-headers

PA Updated by Philippe Antoine over 1 year ago Actions
Copy link
 #10

  • Blocked by Bug #7462: suricata-verify: pass when compiled with FUZZING_UNSAFE added

PA Updated by Philippe Antoine over 1 year ago Actions
Copy link
 #11

  • Status changed from New to In Review

VJ Updated by Victor Julien over 1 year ago Actions
Copy link
 #12

  • Subject changed from pass all suricata-verify tests to suricata-verify: pass all tests

PA Updated by Philippe Antoine about 1 year ago Actions
Copy link
 #13

I guess in the end we also need
- some CI check that we run all SV tests in some config
- check that version requires are not too much (by running with force option)

PA Updated by Philippe Antoine about 1 year ago Actions
Copy link
 #14

  • Target version changed from 8.0.0-beta1 to 8.0.0-rc1

PA Updated by Philippe Antoine about 1 year ago Actions
Copy link
 #15

@Jason Ish did you add a CI check with disable-ja3 ?

PA Updated by Philippe Antoine about 1 year ago Actions
Copy link
 #16

Running master with ./configure --enable-debug --enable-geoip I get the following SV tests that are not run

- versions check : https://github.com/OISF/suricata-verify/pull/2375 makes some more tests run on 8
- requires non-root user : bug-5198, filestore-v2.3-fserror, pcap-log-lz4-write : I checked them manually
- tls-ja3s-requires-off: SKIPPED: not for feature HAVE_JA3 and ja4-rules-requires-off: SKIPPED: not for feature HAVE_JA4, will be run by https://github.com/OISF/suricata/pull/12849
- feature UNITTESTS

PA Updated by Philippe Antoine about 1 year ago Actions
Copy link
 #17

  • Status changed from In Review to Resolved

https://github.com/OISF/suricata/pull/12927

I think we are now complete.

Remaining question is how we remain complete...

VJ Updated by Victor Julien 10 months ago Actions
Copy link
 #18

  • Status changed from Resolved to Closed
Actions
Copy link

Also available in: PDF Atom