From b696f72cf0d1238f8679863c459c24336917c149 Mon Sep 17 00:00:00 2001 From: Seth Grover Date: Mon, 30 Oct 2023 14:53:34 -0600 Subject: [PATCH 1/2] Fix use of deprecated event prototypes (modbus_read_file_record_request, modbus_read_file_record_response, modbus_write_file_record_request, modbus_write_file_record_response) --- scripts/main.zeek | 24 ++++++++++++++++-------- 1 file changed, 16 insertions(+), 8 deletions(-) diff --git a/scripts/main.zeek b/scripts/main.zeek index ad12000..2f71361 100644 --- a/scripts/main.zeek +++ b/scripts/main.zeek @@ -757,8 +757,10 @@ event modbus_read_write_multiple_registers_response(c: connection, ############################################################################################################################# ##################### Defines logging of modbus_read_file_record_request event -> modbus_detailed.log ##################### ############################################################################################################################# -event modbus_read_file_record_request(c: connection, - headers: ModbusHeaders) { +event modbus_read_file_record_request(c: connection, + headers: ModbusHeaders, + byte_count: count, + refs: ModbusFileRecordRequests) { local read_file_record_request: Modbus_Detailed; @@ -782,8 +784,10 @@ event modbus_read_file_record_request(c: connection, ############################################################################################################################# #################### Defines logging of modbus_read_file_record_response event -> modbus_detailed.log ##################### ############################################################################################################################# -event modbus_read_file_record_response(c: connection, - headers: ModbusHeaders) { +event modbus_read_file_record_response(c: connection, + headers: ModbusHeaders, + byte_count: count, + refs: ModbusFileRecordResponses) { local read_file_record_response: Modbus_Detailed; @@ -807,8 +811,10 @@ event modbus_read_file_record_response(c: connection, ############################################################################################################################# #################### Defines logging of modbus_write_file_record_request event -> modbus_detailed.log ##################### ############################################################################################################################# -event modbus_write_file_record_request(c: connection, - headers: ModbusHeaders){ +event modbus_write_file_record_request(c: connection, + headers: ModbusHeaders, + byte_count: count, + refs: ModbusFileReferences) { local write_file_record_request: Modbus_Detailed; @@ -832,8 +838,10 @@ event modbus_write_file_record_request(c: connection, ############################################################################################################################# ################### Defines logging of modbus_write_file_record_response event -> modbus_detailed.log ##################### ############################################################################################################################# -event modbus_write_file_record_response(c: connection, - headers: ModbusHeaders) { +event modbus_write_file_record_response(c: connection, + headers: ModbusHeaders, + byte_count: count, + refs: ModbusFileReferences) { local write_file_record_response: Modbus_Detailed; From fd4254e7633fe82827e018bee1174b07e1fd7532 Mon Sep 17 00:00:00 2001 From: Seth Grover Date: Mon, 30 Oct 2023 15:00:32 -0600 Subject: [PATCH 2/2] Fix use of deprecated event prototypes (modbus_read_file_record_request, modbus_read_file_record_response, modbus_write_file_record_request, modbus_write_file_record_response), but retain compatibility with earlier versions --- scripts/main.zeek | 32 ++++++++++++++++++++++++++++---- 1 file changed, 28 insertions(+), 4 deletions(-) diff --git a/scripts/main.zeek b/scripts/main.zeek index 2f71361..2ea16c2 100644 --- a/scripts/main.zeek +++ b/scripts/main.zeek @@ -757,10 +757,16 @@ event modbus_read_write_multiple_registers_response(c: connection, ############################################################################################################################# ##################### Defines logging of modbus_read_file_record_request event -> modbus_detailed.log ##################### ############################################################################################################################# +@if (Version::at_least("6.1.0")) event modbus_read_file_record_request(c: connection, headers: ModbusHeaders, byte_count: count, - refs: ModbusFileRecordRequests) { + refs: ModbusFileRecordRequests) +@else +event modbus_read_file_record_request(c: connection, + headers: ModbusHeaders) +@endif +{ local read_file_record_request: Modbus_Detailed; @@ -784,10 +790,16 @@ event modbus_read_file_record_request(c: connection, ############################################################################################################################# #################### Defines logging of modbus_read_file_record_response event -> modbus_detailed.log ##################### ############################################################################################################################# +@if (Version::at_least("6.1.0")) event modbus_read_file_record_response(c: connection, headers: ModbusHeaders, byte_count: count, - refs: ModbusFileRecordResponses) { + refs: ModbusFileRecordResponses) +@else +event modbus_read_file_record_response(c: connection, + headers: ModbusHeaders) +@endif +{ local read_file_record_response: Modbus_Detailed; @@ -811,10 +823,16 @@ event modbus_read_file_record_response(c: connection, ############################################################################################################################# #################### Defines logging of modbus_write_file_record_request event -> modbus_detailed.log ##################### ############################################################################################################################# +@if (Version::at_least("6.1.0")) event modbus_write_file_record_request(c: connection, headers: ModbusHeaders, byte_count: count, - refs: ModbusFileReferences) { + refs: ModbusFileReferences) +@else +event modbus_write_file_record_request(c: connection, + headers: ModbusHeaders) +@endif +{ local write_file_record_request: Modbus_Detailed; @@ -838,10 +856,16 @@ event modbus_write_file_record_request(c: connection, ############################################################################################################################# ################### Defines logging of modbus_write_file_record_response event -> modbus_detailed.log ##################### ############################################################################################################################# +@if (Version::at_least("6.1.0")) event modbus_write_file_record_response(c: connection, headers: ModbusHeaders, byte_count: count, - refs: ModbusFileReferences) { + refs: ModbusFileReferences) +@else +event modbus_write_file_record_response(c: connection, + headers: ModbusHeaders) +@endif +{ local write_file_record_response: Modbus_Detailed;