Mirror/zeek
1
0
Fork 0
mirror of https://github.com/zeek/zeek.git synced 2025-10-02 06:38:20 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

rule-parse: Remove id_to_str() lookup to squelch coverity warning

Browse source 
Coverity didn't like that id_to_str() allocates memory and we didn't
free it. Remote its usage wholesale.
This commit is contained in:
Arne Welzel 2024-08-09 09:48:54 +02:00
parent 371dcdc94e
commit fa9dc159a2
3 changed files with 14 additions and 7 deletions
Download patch file Download diff file Expand all files Collapse all files

7
src/rule-parse.y
View file

@ -196,12 +196,7 @@ rule_attr:
if ( is_event($2) )
current_rule->AddAction(new zeek::detail::RuleActionEvent(nullptr, $2));
else
{
const char *msg = id_to_str($2);
if ( ! zeek::util::streq(msg, "<error>") )
zeek::reporter->Error("Identifier %s is not an event (%s:%d)", id_to_str($2),
current_rule_file, rules_line_number+1);
}
rules_error("identifier is not an event", $2);
}
| TOK_EVENT TOK_IDENT TOK_STRING

4
testing/btest/Baseline/signatures.custom-event-errors/.stderr
View file

@ -5,4 +5,6 @@ warning in <...>/custom-event-errors.zeek, line 9: Wrong number of arguments for
error: wrong event parameters for 'wrong_signature3'
warning in <...>/custom-event-errors.zeek, line 11: Type mismatch in function argument #1. Expected string, got count. (event(state:signature_state, msg:count, data:string))
error: wrong event parameters for 'wrong_signature4'
error: Error in signature (./id.sig:19): unknown script-level identifier (non_existing_event)
error: Error in signature (./id.sig:19): identifier is not an event (non_existing_event)
error: Error in signature (./id.sig:24): identifier is not an event (cat)
error: Error in signature (./id.sig:29): identifier is not an event (ignore_checksums)

10
testing/btest/signatures/custom-event-errors.zeek
View file

@ -23,6 +23,16 @@ signature udp-proto4 {
ip-proto == 17
event non_existing_event
}
signature udp-proto5 {
ip-proto == 17
event cat # builtin function
}
signature udp-proto6 {
ip-proto == 17
event ignore_checksums # variable
}
@TEST-END-FILE
event wrong_signature2(state: signature_state, data: string) { }